tyler.durden
/
compose
mirror of https://github.com/docker/compose.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Do not require refresh token to obtain ACR token. Especially this will make the ACR auto-login feature work with Service Principal login, that does not have a refresh token 

Signed-off-by: Guillaume Tardif <guillaume.tardif@docker.com>
This commit is contained in:
Guillaume Tardif 2020-08-19 13:56:41 +02:00
parent 2bb5614501
commit 04c678b099
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
aci/convert/registry_credentials.go
View File

@ -150,10 +150,9 @@ func (c cliRegistryHelper) autoLoginAcr(registry string) error {
}
data := url.Values{
"grant_type": {"access_token_refresh_token"},
"grant_type": {"access_token"},
"service": {registry},
"tenant": {tenantID},
"refresh_token": {token.RefreshToken},
"access_token": {token.AccessToken},
}
repoAuthURL := fmt.Sprintf("https://%s/oauth2/exchange", registry)
@ -162,7 +161,7 @@ func (c cliRegistryHelper) autoLoginAcr(registry string) error {
return err
}
if res.StatusCode != 200 {
return errors.Errorf("error while renewing access token, status : %s", res.Status)
return errors.Errorf("error while accessing ACR token from Azure login, status : %s", res.Status)
}
bits, err := ioutil.ReadAll(res.Body)
if err != nil {