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

Handle RFC 6164 IPv6 addresses #168

Closed
wants to merge 1 commit into from
Closed

Conversation

dclaisse
Copy link
Contributor

Like RFC 3021, IPv6 defines point-to-point subnets that must
be handled separately, i.e. don't reserve first IP address.

This patch aims to implement this, while refactoring code in
iter_host function to reduce code duplication.

Tests for this feature also added to ensure there is no regression.

Signed-off-by: Damien Claisse d.claisse@criteo.com

Like RFC 3021, IPv6 defines point-to-point subnets that must
be handled separately, i.e. don't reserve first IP address.

This patch aims to implement this, while refactoring code in
iter_host function to reduce code duplication.

Tests for this feature also added to ensure there is no regression.

Signed-off-by: Damien Claisse <d.claisse@criteo.com>
@jstasiak
Copy link
Contributor

Thank you, merged into master as 2984c0a.

@jstasiak jstasiak closed this Jun 21, 2020
@jstasiak
Copy link
Contributor

jstasiak commented Jul 3, 2020

@dclaisse We've decided to temporarily revert this before the 0.8.0 release and the documentation will be amended to let people in advance this behavior is going to change (there's a concern about backwards compatibility and people's codebases breaking). The patch will be reinstated in time for the 0.9.0 release so it won't be lost nor forgotten.

@jstasiak
Copy link
Contributor

This has been finally released in version 0.9.0 today.

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.

None yet

2 participants