Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Greg/private networks #1454

Closed
wants to merge 2 commits into from
Closed

Greg/private networks #1454

wants to merge 2 commits into from

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