-
Notifications
You must be signed in to change notification settings - Fork 413
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
Beginnings of acmeDB interface #496
Conversation
- more generalized NotFound type rather than the nosql one we were using - if the error is not recognized then the logic in create account will break.
Codecov Report
@@ Coverage Diff @@
## master #496 +/- ##
==========================================
+ Coverage 77.19% 77.99% +0.80%
==========================================
Files 98 104 +6
Lines 9721 9549 -172
==========================================
- Hits 7504 7448 -56
+ Misses 1811 1725 -86
+ Partials 406 376 -30
Continue to review full report at Codecov.
|
- still needs unit test
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.
Partial review, I still need to finish it but here are some changes.
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.
Hi @dopey just a few details.
- always URL escape linker output - validateJWS should accept RSAPSS - GetUpdateAccount -> GetOrUpdateAccount
acme/db/nosql/nosql.go
Outdated
type Clock int | ||
|
||
// Now returns the UTC time rounded to seconds. | ||
func (c *Clock) Now() time.Time { | ||
return time.Now().UTC().Round(time.Second) | ||
} | ||
|
||
var clock = new(Clock) |
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.
Empty struct and Truncate instead of Round.
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, but I've just added one not on the Clock for nosql.
- NegotiatedProtocolIsMutual is always true: Deprecated according to golang docs
No description provided.