Skip to content

Re-land: Handle RFC 6164 IPv6 addresses#267

Merged
jstasiak merged 1 commit intomasterfrom
restore-a-fix
Sep 14, 2023
Merged

Re-land: Handle RFC 6164 IPv6 addresses#267
jstasiak merged 1 commit intomasterfrom
restore-a-fix

Conversation

@jstasiak
Copy link
Contributor

This is a cherry pick of commit [1] later temporarily reverted in [2] to unblock the 0.8.0 release.

The original commit message:

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>

We're preparing 0.9.0 release so this can land again (for good this time).

[1] 2984c0a ("Handle RFC 6164 IPv6 addresses")
[2] 67f1992 ("Revert "Handle RFC 6164 IPv6 addresses"")

This is a cherry pick of commit [1] later temporarily reverted in [2] to
unblock the 0.8.0 release.

The original commit message:

    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>

We're preparing 0.9.0 release so this can land again (for good this
time).

[1] 2984c0a ("Handle RFC 6164 IPv6 addresses")
[2] 67f1992 ("Revert "Handle RFC 6164 IPv6 addresses"")
@codecov-commenter
Copy link

codecov-commenter commented Sep 14, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.02% 🎉

Comparison is base (f09c3b2) 85.32% compared to head (49ea726) 85.34%.
Report is 6 commits behind head on master.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #267      +/-   ##
==========================================
+ Coverage   85.32%   85.34%   +0.02%     
==========================================
  Files          48       48              
  Lines        4566     4573       +7     
  Branches      706      805      +99     
==========================================
+ Hits         3896     3903       +7     
  Misses        488      488              
  Partials      182      182              
Files Changed Coverage Δ
netaddr/tests/ip/test_ip_v4.py 99.24% <ø> (-0.02%) ⬇️
netaddr/ip/__init__.py 81.08% <100.00%> (+0.02%) ⬆️
netaddr/tests/ip/test_ip_v6.py 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jstasiak jstasiak merged commit c2b6491 into master Sep 14, 2023
@jstasiak jstasiak deleted the restore-a-fix branch September 14, 2023 23:41
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.

3 participants