Skip to content

Commit

Permalink
No need to force https (olivere#1326)
Browse files Browse the repository at this point in the history
There is no need to force https in the aws signer.
This already happens by default, and forcing it ruins the ability to test in dev against an Elasticsearch docker container.

Close olivere#1328
  • Loading branch information
veqryn authored May 9, 2020
1 parent e9b399f commit 639b7f6
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion aws/v4/aws_v4.go
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,6 @@ func (st Transport) RoundTrip(req *http.Request) (*http.Response, error) {
return st.client.Do(req)
}

req.URL.Scheme = "https"
if strings.Contains(req.URL.RawPath, "%2C") {
// Escaping path
req.URL.RawPath = url.PathEscape(req.URL.RawPath)
Expand Down

0 comments on commit 639b7f6

Please sign in to comment.