Skip to content

Conversation

@abitfan
Copy link
Contributor

@abitfan abitfan commented Apr 17, 2019

Improves IP diversity for hardcoded seednodes.

@DrahtBot
Copy link
Contributor

DrahtBot commented Apr 18, 2019

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts

No conflicts as of last run.

@laanwj
Copy link
Member

laanwj commented Apr 18, 2019

Concept ACK

1 similar comment
@practicalswift
Copy link
Contributor

Concept ACK

@laanwj
Copy link
Member

laanwj commented May 1, 2019

Tested ACK on the query part

one question though before I merge: are the IPv4 and IPv6 ASNs in a single namespace? if not, asn_count logic should take this into account

@laanwj laanwj requested a review from sipa May 1, 2019 11:53
@abitfan
Copy link
Contributor Author

abitfan commented May 1, 2019

Yes, they are in the same namespace (notice ipv46) only max_per_asn will be chosen, weather they're all v4 or v6 or a combination of v4+v6

@laanwj laanwj merged commit 316b8b2 into bitcoin:master May 20, 2019
laanwj added a commit that referenced this pull request May 20, 2019
316b8b2 Filter IPv6 by ASN (Emil)

Pull request description:

  Improves IP diversity for hardcoded seednodes.

ACKs for commit 316b8b:

Tree-SHA512: ae90427efa317d59125457bf8bfd077fd115c0921e1cc13cebd855206498546a026ccc18f039d1963d64d9be9497c41f4a21214fb565d5d11a9635ad12836421
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request May 20, 2019
316b8b2 Filter IPv6 by ASN (Emil)

Pull request description:

  Improves IP diversity for hardcoded seednodes.

ACKs for commit 316b8b:

Tree-SHA512: ae90427efa317d59125457bf8bfd077fd115c0921e1cc13cebd855206498546a026ccc18f039d1963d64d9be9497c41f4a21214fb565d5d11a9635ad12836421
@laanwj laanwj mentioned this pull request Sep 30, 2019
Munkybooty pushed a commit to Munkybooty/dash that referenced this pull request Nov 18, 2021
316b8b2 Filter IPv6 by ASN (Emil)

Pull request description:

  Improves IP diversity for hardcoded seednodes.

ACKs for commit 316b8b:

Tree-SHA512: ae90427efa317d59125457bf8bfd077fd115c0921e1cc13cebd855206498546a026ccc18f039d1963d64d9be9497c41f4a21214fb565d5d11a9635ad12836421
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Dec 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants