Skip to content

Greg/private networks#1454

Closed
greg-szabo wants to merge 2 commits intodevelopfrom
greg/private-networks
Closed

Greg/private networks#1454
greg-szabo wants to merge 2 commits intodevelopfrom
greg/private-networks

Conversation

@greg-szabo
Copy link
Contributor

Enabled the use of private IPv4 addresses. (RFC1918). This is required to run tendermint on a private network where all the nodes are on the same private network.

@codecov-io
Copy link

Codecov Report

Merging #1454 into develop will decrease coverage by 0.02%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #1454      +/-   ##
===========================================
- Coverage    63.67%   63.64%   -0.03%     
===========================================
  Files          102      102              
  Lines         9459     9452       -7     
===========================================
- Hits          6023     6016       -7     
- Misses        2871     2873       +2     
+ Partials       565      563       -2
Impacted Files Coverage Δ
p2p/netaddress.go 69.18% <100%> (-2.52%) ⬇️
blockchain/pool.go 68.05% <0%> (-0.7%) ⬇️
p2p/pex/pex_reactor.go 70.27% <0%> (-0.35%) ⬇️
p2p/peer.go 63.76% <0%> (ø) ⬆️
consensus/reactor.go 72.59% <0%> (+0.26%) ⬆️
consensus/state.go 77.06% <0%> (+0.27%) ⬆️
consensus/replay.go 69.89% <0%> (+0.46%) ⬆️

@ebuchman
Copy link
Contributor

@greg-szabo have you seen the "addr_book_strict" option in the config ?

# Set true for strict address routability rules
addr_book_strict = true

Does that work ? Maybe we can expose it as a flag if need be ?

Not sure we want to just remove this kind of check out right

@melekes
Copy link
Contributor

melekes commented Apr 20, 2018

#1455 (comment)

@greg-szabo
Copy link
Contributor Author

I wrote a long dissection of the RFCs on the issue. All in all, I agree: we should remove this kind of check outright.

@greg-szabo
Copy link
Contributor Author

Closing it, as agreed on the meeting.

@greg-szabo greg-szabo closed this Apr 24, 2018
@melekes melekes deleted the greg/private-networks branch April 24, 2018 15:18
melekes added a commit that referenced this pull request Apr 24, 2018
melekes added a commit that referenced this pull request Apr 24, 2018
melekes added a commit that referenced this pull request Apr 26, 2018
* [cmd] Turn off strict routability when using testnet cmd

Refs #1454

* use "testnet" instead of "it"
firelizzard18 pushed a commit to AccumulateNetwork/tendermint that referenced this pull request Feb 1, 2024
)

Bumps [golang.org/x/sync](https://github.com/golang/sync) from 0.3.0 to 0.4.0.
- [Commits](golang/sync@v0.3.0...v0.4.0)

---
updated-dependencies:
- dependency-name: golang.org/x/sync
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants