diff --git a/CHANGELOG.md b/CHANGELOG.md index f0d68fcae..e40d35472 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,5 +1,11 @@ # CHANGELOG +## v14.1.1 + +### Bug Fixes + +- Change `x-ms-authorization-auxiliary` header value separator to comma. + ## v14.1.0 ### New Features diff --git a/autorest/authorization.go b/autorest/authorization.go index f43e1a6ed..5802f6a4b 100644 --- a/autorest/authorization.go +++ b/autorest/authorization.go @@ -331,7 +331,7 @@ func (mt multiTenantSPTAuthorizer) WithAuthorization() PrepareDecorator { for i := range auxTokens { auxTokens[i] = fmt.Sprintf("Bearer %s", auxTokens[i]) } - return Prepare(r, WithHeader(headerAuxAuthorization, strings.Join(auxTokens, "; "))) + return Prepare(r, WithHeader(headerAuxAuthorization, strings.Join(auxTokens, ", "))) }) } } diff --git a/autorest/authorization_test.go b/autorest/authorization_test.go index 2df8b9b8f..18f1cc165 100644 --- a/autorest/authorization_test.go +++ b/autorest/authorization_test.go @@ -299,7 +299,7 @@ func TestMultitenantAuthorizationThree(t *testing.T) { if primary := req.Header.Get(headerAuthorization); primary != "Bearer primary" { t.Fatalf("bad primary authorization header %s", primary) } - if aux := req.Header.Get(headerAuxAuthorization); aux != "Bearer aux1; Bearer aux2; Bearer aux3" { + if aux := req.Header.Get(headerAuxAuthorization); aux != "Bearer aux1, Bearer aux2, Bearer aux3" { t.Fatalf("bad auxiliary authorization header %s", aux) } } @@ -376,7 +376,7 @@ func TestMultiTenantServicePrincipalTokenWithAuthorizationRefresh(t *testing.T) auxTokens[i] = fmt.Sprintf("Bearer %s", auxTokens[i]) } auxHeader := req.Header.Get(http.CanonicalHeaderKey(headerAuxAuthorization)) - if auxHeader != strings.Join(auxTokens, "; ") { + if auxHeader != strings.Join(auxTokens, ", ") { t.Fatal("azure: multiTenantSPTAuthorizer#WithAuthorization failed to set Authorization header for auxiliary tokens") } for i := range auxTokens { diff --git a/autorest/version.go b/autorest/version.go index ee6e48f8b..d81116ccc 100644 --- a/autorest/version.go +++ b/autorest/version.go @@ -19,7 +19,7 @@ import ( "runtime" ) -const number = "v14.1.0" +const number = "v14.1.1" var ( userAgent = fmt.Sprintf("Go/%s (%s-%s) go-autorest/%s", diff --git a/azure-pipelines.yml b/azure-pipelines.yml index d283dd92e..6fb8404fd 100644 --- a/azure-pipelines.yml +++ b/azure-pipelines.yml @@ -11,23 +11,25 @@ jobs: Linux_Go113: vm.image: 'ubuntu-18.04' go.version: '1.13' - GOROOT: '/usr/local/go$(go.version)' Linux_Go114: vm.image: 'ubuntu-18.04' go.version: '1.14' - GOROOT: '/usr/local/go$(go.version)' pool: vmImage: '$(vm.image)' steps: + - task: GoTool@0 + inputs: + version: '$(go.version)' + displayName: "Select Go Version" + - script: | set -e mkdir -p '$(GOPATH)/bin' mkdir -p '$(sdkPath)' shopt -s extglob mv !(work) '$(sdkPath)' - echo '##vso[task.prependpath]$(GOROOT)/bin' echo '##vso[task.prependpath]$(GOPATH)/bin' displayName: 'Create Go Workspace'