Keep compatibility with Go 1.13.14. (#264)

* - Keep compliance with Golang 1.13.14. http.Header has no method Values
  on older go versions.

* Do not return error when client request id is missing
This commit is contained in:
Sreejith Kesavan 2021-03-17 21:56:11 +05:30 коммит произвёл GitHub
Родитель b0f228e10f
Коммит 5e5d32824b
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: 4AEE18F83AFDEB23
2 изменённых файлов: 8 добавлений и 8 удалений

Просмотреть файл

@ -3,6 +3,7 @@ package azblob
import (
"context"
"errors"
"github.com/Azure/azure-pipeline-go/pipeline"
)
@ -21,11 +22,9 @@ func NewUniqueRequestIDPolicyFactory() pipeline.Factory {
resp, err := next.Do(ctx, request)
if err == nil && resp != nil {
val := resp.Response().Header.Values(xMsClientRequestID)
if len(val) > 0 {
if val[0] != id {
err = errors.New("client Request ID from request and response does not match")
}
crId := resp.Response().Header.Get(xMsClientRequestID)
if crId != "" && crId != id {
err = errors.New("client Request ID from request and response does not match")
}
}

Просмотреть файл

@ -3,10 +3,11 @@ package azblob
import (
"context"
"errors"
"github.com/Azure/azure-pipeline-go/pipeline"
chk "gopkg.in/check.v1"
"net/http"
"net/url"
"github.com/Azure/azure-pipeline-go/pipeline"
chk "gopkg.in/check.v1"
)
type requestIDTestScenario int
@ -58,7 +59,7 @@ func (s *aztestsSuite) TestEchoClientRequestIDMissing(c *chk.C) {
c.Assert(err, chk.IsNil)
c.Assert(resp, chk.NotNil)
c.Assert(resp.Response().Header.Values(xMsClientRequestID), chk.IsNil)
c.Assert(resp.Response().Header.Get(xMsClientRequestID), chk.Equals, "")
}
func (s *aztestsSuite) TestEchoClientRequestIDErrorFromNextPolicy(c *chk.C) {