forked from TrueCloudLab/distribution
Merge pull request #10471 from jlhawn/token_auth_header
Fix token basic auth header issue
This commit is contained in:
commit
cbda817ec3
2 changed files with 6 additions and 2 deletions
|
@ -128,6 +128,8 @@ func (r *Session) HeadV2ImageBlob(ep *Endpoint, imageName, sumType, sum string,
|
||||||
case res.StatusCode >= 200 && res.StatusCode < 400:
|
case res.StatusCode >= 200 && res.StatusCode < 400:
|
||||||
// return something indicating no push needed
|
// return something indicating no push needed
|
||||||
return true, nil
|
return true, nil
|
||||||
|
case res.StatusCode == 401:
|
||||||
|
return false, errLoginRequired
|
||||||
case res.StatusCode == 404:
|
case res.StatusCode == 404:
|
||||||
// return something indicating blob push needed
|
// return something indicating blob push needed
|
||||||
return false, nil
|
return false, nil
|
||||||
|
|
|
@ -51,10 +51,12 @@ func getToken(username, password string, params map[string]string, registryEndpo
|
||||||
reqParams.Add("scope", scopeField)
|
reqParams.Add("scope", scopeField)
|
||||||
}
|
}
|
||||||
|
|
||||||
reqParams.Add("account", username)
|
if username != "" {
|
||||||
|
reqParams.Add("account", username)
|
||||||
|
req.SetBasicAuth(username, password)
|
||||||
|
}
|
||||||
|
|
||||||
req.URL.RawQuery = reqParams.Encode()
|
req.URL.RawQuery = reqParams.Encode()
|
||||||
req.SetBasicAuth(username, password)
|
|
||||||
|
|
||||||
resp, err := client.Do(req)
|
resp, err := client.Do(req)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
|
|
Loading…
Reference in a new issue