This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: IPv6Network constructor sometimes does not recognize legitimate netmask
Type: enhancement Stage:
Components: Library (Lib) Versions: Python 3.5
process
Status: closed Resolution: wont fix
Dependencies: Superseder:
Assigned To: Nosy List: ncoghlan, pebenito, pitrou, pmoody
Priority: normal Keywords:

Created on 2014-11-05 15:34 by pebenito, last changed 2022-04-11 14:58 by admin. This issue is now closed.

Messages (8)
msg230686 - (view) Author: Chris PeBenito (pebenito) Date: 2014-11-05 15:34
Python 3.3/3.4 sometimes does not recognize a legitimate IPv6Network netmask:

$ python3
Python 3.3.5 (default, May 28 2014, 13:56:57) 
[GCC 4.7.3] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import ipaddress as ip
>>> nodecon = ip.IPv6Network('ff00::/ff00::')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib64/python3.3/ipaddress.py", line 2084, in __init__
    self._prefixlen = self._prefix_from_prefix_string(addr[1])
  File "/usr/lib64/python3.3/ipaddress.py", line 514, in _prefix_from_prefix_string
    self._report_invalid_netmask(prefixlen_str)
  File "/usr/lib64/python3.3/ipaddress.py", line 497, in _report_invalid_netmask
    raise NetmaskValueError(msg) from None
ipaddress.NetmaskValueError: 'ff00::' is not a valid netmask
>>> nodecon = ip.IPv6Network('ff00::/8')
>>> print(nodecon)
ff00::/8
>>> print(nodecon.with_netmask)
ff00::/ff00::

I get the same behavior on Python 3.4.2.
msg230868 - (view) Author: Antoine Pitrou (pitrou) * (Python committer) Date: 2014-11-08 16:25
The doc is unhelpful on this, but looking at the implementation and tests, only a prefix length is allowed, not an expanded netmask. This would therefore be a feature request.
msg230869 - (view) Author: Chris PeBenito (pebenito) Date: 2014-11-08 17:32
That's unfortunate. The library provides factory functions so v4 and v6 addresses/networks are easily handled together, and yet it seems to have been overlooked that you can do this:

ipaddress.ip_network('192.168.1.0/255.255.255.0') 

but not this:

ipaddress.ip_network('ff00::/ff00::')

I'll open up another issue for the docs, as they are not simply unhelpful, they're misleading, as the IPv6Network docs clearly say that you should be able to use an expanded netmask.
msg230870 - (view) Author: Antoine Pitrou (pitrou) * (Python committer) Date: 2014-11-08 17:33
I don't know enough about IPv6 to give more insight (perhaps Peter Moody can answer), but the tests have this comment: 

# We only support CIDR for IPv6, because expanded netmasks are not
# standard notation.
msg230969 - (view) Author: pmoody (pmoody) * (Python committer) Date: 2014-11-10 18:44
> # We only support CIDR for IPv6, because expanded netmasks are not
> # standard notation.

Yes, that's correct. I can double check this, but when I wrote ipaddress, I had yet to encounter a v6 netmask in anything other than cider notation.
msg230970 - (view) Author: pmoody (pmoody) * (Python committer) Date: 2014-11-10 18:52
Hey Chris,

What's the usecase for this? the netmask notation doesn't appear to be common for v6 (at all), so I'm hesitant to add support for this if it's just something like an academic exercise.

Cheers,
peter
msg230971 - (view) Author: Chris PeBenito (pebenito) Date: 2014-11-10 20:08
I understand the resistance; I'm fine closing this as "won't implement", though this is not for academic use.  In a nutshell, my package currently has a set of classes to represent an SELinux policy, and the SELinux policy language represents networks with extended netmask[1].  I control the package, so I can change the internal representation from extended netmask to CIDR, so it's not the end of the world.

[1] example statement: nodecon ff00:: ff00:: system_u:object_r:multicast_node_t
msg230973 - (view) Author: pmoody (pmoody) * (Python committer) Date: 2014-11-10 20:54
If you have the ability to use cidr, then closing this as wontfix is my preference. I've heard that there might be some network vendors that are starting support the mask notation for v6 addresses though, so this may end up getting implemented at some point in future anyway.
History
Date User Action Args
2022-04-11 14:58:09adminsetgithub: 66989
2014-11-10 20:54:42pmoodysetstatus: open -> closed
resolution: wont fix
messages: + msg230973
2014-11-10 20:08:25pebenitosetmessages: + msg230971
2014-11-10 18:52:43pmoodysetmessages: + msg230970
2014-11-10 18:44:54pmoodysetmessages: + msg230969
2014-11-08 17:33:58pitrousetmessages: + msg230870
2014-11-08 17:32:04pebenitosetmessages: + msg230869
2014-11-08 16:25:59pitrousettype: behavior -> enhancement
messages: + msg230868
versions: - Python 3.4
2014-11-08 15:23:16pitrousetversions: + Python 3.5, - Python 3.3
2014-11-05 20:41:41rhettingersetnosy: + pitrou
2014-11-05 20:34:38ned.deilysetnosy: + ncoghlan, pmoody
2014-11-05 15:34:45pebenitocreate