-
Notifications
You must be signed in to change notification settings - Fork 27
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
multi: various improvements #297
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.
Apologies for the delay on this, xmas etc.
A couple of minor things
return nil, nil, err | ||
} | ||
|
||
// Create the data directory. | ||
err = os.MkdirAll(cfg.DataDir, 0700) | ||
if err != nil { | ||
str := "%s: failed to create data directory: %v" | ||
err := fmt.Errorf(str, funcName, err) | ||
str := "%s: unable to create data directory (%s): %v" |
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.
funcName
is used in a handful of places, but not all. Any reason? I guess we could remove it completely
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 think it makes sense to keep it for config related errors resulting from creating directories, generating certificates etc which are not directly related to the config process.Errors related to config parsing and parameter constraints can do without the func name label since those indicate an app-wide config issue that's why funcName
is used only in some places.
e196c92
to
010dd5f
Compare
This adds additional checks for the --pool config to ensure the supplied value is a valid address and the miner is not started without one.
This adds additional checks for postgres config options and improves config error outputs.
This removes the letsencrypt http redirect because clients should be expecting an https service and if there is a need to for a redirect the pool operator's web server should be capable.
This fixes a bug where solo pool gui where not updating mined work properly because the cache was not being properly updated on block connections. This fix also moe efficiently confirms work.
This refactors the prrocess of establishing a node connection and a wallet connection to avoid the use of setters for the connections.
This adds example outputs for pool startups for solo pool mode and public pool mode.
All good after fixing that typo |
Had to push another commit. |
This updates the cpu miner stratum handshake to only send an authorization request after the subscription request has been processed.
This improves config options handling, error outputs and letsencrypt http redirects.