-
Notifications
You must be signed in to change notification settings - Fork 2
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
Consider --bind rather than --port/--address #16
Comments
We could just add |
@spacekookie I don't like the idea of duplicating functionality in what's considered a limited space (e.g. there's a thing like too many flags). Instead I think it might be better to create a separate module called |
@yoshuawuyts TBH that's how I was reading @spacekookie's suggestion (although I was thinking in the same crate). That way it's up to the developer of the service to choose how they want to present it to their users. |
I originally had a feature flag in mind, but creating a |
I realise I'm going against a lot of prior server CLI design, but I much prefer a more descriptive command line argument than the normal
--port
argument. A design that I've used in the past iswhich can reuse
SocketAddr::from_str
for parsing.The text was updated successfully, but these errors were encountered: