Skip to content

Conversation

shawkins
Copy link
Contributor

@shawkins shawkins commented Sep 30, 2025

The bind address 0.0.0.0 for all addresses should not be shown in the logs in a url.

Previously one might see:

Listening on: https://0.0.0.0:8444

An initial proprosal with this pr would show:

Listening on: all addresses on port 8444 for https (e.g. https://127.0.0.1:8444)

This was refined to be less verbose:

Listening on: all addresses including https://localhost:8444

Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

Let's not rush this as it's our startup message.

Can you please add in the description some examples of the results, compared to how it was before.

Because it looks very verbose.

Thanks!

@shawkins
Copy link
Contributor Author

shawkins commented Sep 30, 2025

@gsmet @geoand tried to make it less verbose and to handle the ipv6 cases as well.

Let me know if you have any other thoughts on how to convey this susinctly.

@geoand
Copy link
Contributor

geoand commented Sep 30, 2025

Can you please add in the description some examples of the results, compared to how it was before.

Yeah, this would be really nice to see

@shawkins
Copy link
Contributor Author

Yeah, this would be really nice to see

The description reflects the latest changes.

@geoand
Copy link
Contributor

geoand commented Oct 1, 2025

I like the latest incarnation of this

serverListeningMessage.append(" and ");
}
serverListeningMessage.append(String.format("unix:%s", getDeveloperFriendlyHostName(domainSocketOptions)));
serverListeningMessage.append(String.format("unix:%s", domainSocketOptions.getHost()));
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Domain sockets don't understand the any address concept, so we don't need special handling here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Do not show using 0.0.0.0 in a url
3 participants