-
Notifications
You must be signed in to change notification settings - Fork 83
Conversation
/hold |
go.mod
Outdated
@@ -2,6 +2,8 @@ module github.com/google/exposure-notifications-verification-server | |||
|
|||
go 1.14 | |||
|
|||
replace github.com/google/exposure-notifications-server => ../exposure-notifications-server |
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.
Will need to update this after merging google/exposure-notifications-server#957
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.
pls update to v0.7.0
@@ -89,25 +89,6 @@ func (c *Config) Load(ctx context.Context) (*Database, error) { | |||
logger.Errorf("key manager does not support the SigningKeyManager interface, falling back to single verification signing key") | |||
} | |||
|
|||
// If the key manager is in-memory, accept the key as a base64-encoded |
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'm glad to get rid of this - it was... hacky.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mikehelmick, sethvargo The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Depends on google/exposure-notifications-server#957
Release Note