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
To go cdns sslkeys #2400
To go cdns sslkeys #2400
Conversation
Refer to this link for build results (access rights to CI server needed): |
d4327cf
to
390fb5a
Compare
Refer to this link for build results (access rights to CI server needed): |
390fb5a
to
2f6bb87
Compare
Refer to this link for build results (access rights to CI server needed): |
2f6bb87
to
7e93215
Compare
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.
This might could use some improvement, but it appears to do what the Perl did, for better or for worse. I suggest merging and then reconsidering the things noted in the TODOs.
Refer to this link for build results (access rights to CI server needed): |
7e93215
to
041833d
Compare
Refer to this link for build results (access rights to CI server needed): |
041833d
to
a8e22fe
Compare
Refer to this link for build results (access rights to CI server needed): |
a8e22fe
to
c103bdb
Compare
Refer to this link for build results (access rights to CI server needed): |
c103bdb
to
05bcfaa
Compare
Refer to this link for build results (access rights to CI server needed): |
05bcfaa
to
82b61ee
Compare
@@ -108,6 +108,7 @@ func Routes(d ServerData) ([]Route, []RawRoute, http.Handler, error) { | |||
{1.1, http.MethodPost, `cachegroups/{id}/queue_update$`, cachegroup.QueueUpdates(d.DB.DB), auth.PrivLevelOperations, Authenticated, nil}, | |||
|
|||
//CDN | |||
{1.1, http.MethodGet, `cdns/name/{name}/sslkeys/?(\.json)?$`, cdn.GetSSLKeys, auth.PrivLevelOperations, Authenticated, nil}, |
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 this is supposed to be admin priv level
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.
@mitchell852 Fixed
82b61ee
to
b81c86b
Compare
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
Branched from #2365 - recommend merging that first.