Skip to content

Remove misleading parts of is_site_local() documentation#113152

Open
jstasiak wants to merge 2 commits intopython:mainfrom
jstasiak:remove-misleading-documentation
Open

Remove misleading parts of is_site_local() documentation#113152
jstasiak wants to merge 2 commits intopython:mainfrom
jstasiak:remove-misleading-documentation

Conversation

@jstasiak
Copy link
Copy Markdown
Contributor

@jstasiak jstasiak commented Dec 15, 2023

In case of IPv6Address and IPv6Network is_private is very much not an equivalent or a replacement of is_site_local().

IPv6 is_private considers many ranges other than RFC 4193's fc00::/7 to be private (see _IPv6Constants._private_networks for the complete list).

IPv6Address.is_private goes even further and delegates to IPv4Address.is_private in case of an IPv4-mapped IPv6 address.


📚 Documentation preview 📚: https://cpython-previews--113152.org.readthedocs.build/

In case of IPv6Address and IPv6Network is_private is very much not an
equivalent or a replacement of is_site_local().

IPv6 is_private considers many ranges other than RFC 4193's fc00::/7 to
be private (see _IPv6Constants._private_networks for the complete list).

IPv6Address.is_private goes even further and delegates to
IPv4Address.is_private in case of an IPv4-mapped IPv6 address.
@jstasiak
Copy link
Copy Markdown
Contributor Author

Seems like a skip issue + skip news change to me but I'll do what's needed if requested.

@github-actions
Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review docs Documentation in the Doc dir skip issue skip news stale Stale PR or inactive for long period of time.

Projects

Status: Todo
Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants