Skip to content

log failure to start service (e.g., port already in use)#31

Open
zackse wants to merge 1 commit into
cllunsford:masterfrom
zackse:log-failure-to-bind
Open

log failure to start service (e.g., port already in use)#31
zackse wants to merge 1 commit into
cllunsford:masterfrom
zackse:log-failure-to-bind

Conversation

@zackse

@zackse zackse commented Apr 19, 2019

Copy link
Copy Markdown

Previously, failure to bind would result in a silent failure after the "Listening on " message. This change wraps the call to ListenAndServe to report any failure.

# ./aws-signing-proxy -target https://my-endpoint.amazonaws.com -port 9999 &
[3] 26203
Listening on :9999

# ./aws-signing-proxy -target https://my-endpoint.amazonaws.com -port 9999
Listening on :9999
2019/04/19 11:51:43 listen tcp :9999: bind: address already in use

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