#17400: fix documentation, add cache to is_global and correctly handle 100.64.0.0/10

This commit is contained in:
Peter Moody 2013-10-22 12:36:21 -07:00
parent a4df90ceb9
commit be9c1b133b
3 changed files with 7 additions and 4 deletions

View File

@ -160,7 +160,7 @@ write code that handles both IP versions correctly.
.. attribute:: is_global
``True`` if the address is allocated for private networks. See
``True`` if the address is allocated for public networks. See
iana-ipv4-special-registry (for IPv4) or iana-ipv6-special-registry
(for IPv6).

View File

@ -984,7 +984,7 @@ class _BaseNetwork(_IPAddressBase):
@property
def is_global(self):
"""Test if this address is allocated for private networks.
"""Test if this address is allocated for public networks.
Returns:
A boolean, True if the address is not reserved per
@ -1233,6 +1233,7 @@ class IPv4Address(_BaseV4, _BaseAddress):
return self in reserved_network
@property
@functools.lru_cache()
def is_private(self):
"""Test if this address is allocated for private networks.
@ -1259,14 +1260,14 @@ class IPv4Address(_BaseV4, _BaseAddress):
@property
def is_global(self):
"""Test if this address is allocated for private networks.
"""Test if this address is allocated for public networks.
Returns:
A boolean, True if the address is not reserved per
iana-ipv4-special-registry.
"""
return not self.is_private
return self in IPv4Network('100.64.0.0/10') or not self.is_private
@property
@ -1856,6 +1857,7 @@ class IPv6Address(_BaseV6, _BaseAddress):
return self in sitelocal_network
@property
@functools.lru_cache()
def is_private(self):
"""Test if this address is allocated for private networks.

View File

@ -1320,6 +1320,7 @@ class IpaddrUnitTest(unittest.TestCase):
'127.42.0.0/16').is_loopback)
self.assertEqual(False, ipaddress.ip_network('128.0.0.0').is_loopback)
self.assertEqual(True, ipaddress.ip_network('100.64.0.0/10').is_private)
self.assertEqual(False, ipaddress.ip_network('100.64.0.0/10').is_global)
self.assertEqual(True,
ipaddress.ip_network('192.0.2.128/25').is_private)
self.assertEqual(True,