-
-
Notifications
You must be signed in to change notification settings - Fork 7.4k
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
Make bind interface configurable, defaulting to 127.0.0.1 #1099
Conversation
Thanks for this. I'm not sure why I would want to do the below, but the log statement "Web Server is ..." is falsy below. Could you fix that (amend the commit) -- and at the same time improve your commit message: split it in a title and body and end it with "Fixes #theid".
|
@bep Glad I can help! What is the solution to your conflict there? If "bind" is specified, it is likely to be "0.0.0.0" meaning "all interfaces." Of course the base URL should not be set to 0.0.0.0, right? But at the same time, we can't set the bind address to the base URL, since the base URL could contain components other than an address. So when there's a conflict between bind address and base URL, who "wins?" |
The solution is to fix the log message. Should always use the value in bind. |
TIL that 0.0.0.0 can be used to connect to the local machine. Did not know that :) Updating patch now. |
@hut8 I don't see any updated patch ...? |
Bind by default to 127.0.0.1. Previously it always bound to all interfaces. Always have log message display that the server is available on the interface to which it is bound, instead of the baseUrl. Fixes gohugoio#1097
Hang on, squashing commits, fixing commit messages and pushing... |
OK, just ping me when done ... |
I forgot to reply to that before. I pushed the commits a while ago. Should be good now. |
Nah, my fault -- I looked at the changes, but didn't notice that part ... All good. Merged. Thanks. |
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Previously the server would always bind to all interfaces, which is weird since it's usually for testing. Resolves #1097 - also resolves some passive voice in help strings.