vbatts/registry
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Only do auth checks for endpoints starting with v2

Browse source 
Changes behavior so ping doesn't happen if /v2/ is anywhere in a request path, but instead only at the beginning. This fixes attempts to ping on redirected URLs.

Signed-off-by: Derek McGowan <derek@mcgstyle.net> (github: dmcgowan)
This commit is contained in:
Derek McGowan 2015-05-20 13:35:23 -07:00
parent c7f7747368
commit 49369ffe9a
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
registry/client/transport/session.go
View file

@ -94,7 +94,9 @@ HeaderLoop:
func (ta *tokenAuthorizer) ModifyRequest(req *http.Request) error { func (ta *tokenAuthorizer) ModifyRequest(req *http.Request) error {
v2Root := strings.Index(req.URL.Path, "/v2/") v2Root := strings.Index(req.URL.Path, "/v2/")
if v2Root == -1 { // Test if /v2/ does not exist or not at beginning
// TODO(dmcgowan) support v2 endpoints which have a prefix before /v2/
if v2Root == -1 || v2Root > 0 {
return nil return nil
} }