-
Notifications
You must be signed in to change notification settings - Fork 213
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
server: Listen on unused TCP port selected at random #387
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! There are a couple of minor points that I've addressed in comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Server.start settings wallet | ||
let logStartup port = TIO.hPutStrLn stderr $ | ||
"Wallet backend server listening on: " <> toText port | ||
Server.start logStartup walletPort wallet |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if it wouldn't be slightly "cleaner" to let start
takes a Warp.Settings
as a parameter. But no strong feeling on that.
10e9770
to
151f4c2
Compare
Not sure why the CI check isn't reported to Github but CI has completed for a while already (and is green). Merging this. |
Relates to issue #144
Overview
cardano-wallet server --port
command-line argument is optional.