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

More IPv6: Use bare IPv6 for configuration, use [ipv6] when displaying IPv6 outputs #4988

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

manics
Copy link
Member

@manics manics commented Jan 27, 2025

No description provided.

In most cases, this does not result in a change in behavior,
as '' was interpreted as 'unspecified',
which used the subprocesses' own default, itself usually '127.0.0.1'.
Default changed to '127.0.0.1', from unspecified.
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure whether to update or leave this text. It made sense in the context of the 2.0 upgrade, but coupled with subsequent changes it potentially adds confusion.

In hindsight how we interpret the meaning of '' affects how we treat #4986
If we consider '' == 127.0.0.1 a bug, which is what I was thinking since it's otherwise impossible to pass '' to the singleuser server, then the meaning of '' has changed from unspecified (prior to 2.0) to impossible/invalid (between 2.0 and the current release) to all interfaces (#4986).

However if we consider '' == 127.0.0.1 intentional then #4986 is a breaking change- which we could workaround by making '*' mean all interfaces instead. However this risks adding more confusion since it'll still be converted to '' when passed to the singleuser server.

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.

1 participant