Skip to content

Commit

Permalink
service/finspace: Fix SDK's ContentType sent for requests (#3896)
Browse files Browse the repository at this point in the history
Updates the service/finspace and service/finspacedata to send the expected Content-Type value for requests.

* fixup usage of golang.org/x/net/http2 to be 1.15 and higher
  • Loading branch information
jasdel committed May 5, 2021
1 parent fc28b08 commit 66dee9c
Show file tree
Hide file tree
Showing 105 changed files with 252 additions and 108 deletions.
7 changes: 0 additions & 7 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,13 +49,6 @@ before_install:

script:
- if [ -z $(go env GOMOD) ]; then
if [ "$TRAVIS_GO_VERSION" == "1.14.x" ] ||
[ "$TRAVIS_GO_VERSION" == "1.13.x" ] ||
[ "$TRAVIS_GO_VERSION" == "1.12.x" ] ||
[ "$TRAVIS_GO_VERSION" == "1.11.x" ] ||
[ "$TRAVIS_GO_VERSION" == "1.10.x" ]; then
make get-deps-x-tests;
fi;
make get-deps;
fi;
if [ "$TRAVIS_GO_VERSION" == "tip" ] ||
Expand Down
4 changes: 4 additions & 0 deletions CHANGELOG_PENDING.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,7 @@
### SDK Enhancements

### SDK Bugs
* `service/finspace`: Fix Content-Type header sent in request ([#3896](https://github.com/aws/aws-sdk-go/pull/3896))
* Fixes the Content-Type header sent by the SDK in requests to the API
* `service/finspacedata`: Fix Content-Type header sent in request ([#3896](https://github.com/aws/aws-sdk-go/pull/3896))
* Fixes the Content-Type header sent by the SDK in requests to the API
4 changes: 0 additions & 4 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -227,10 +227,6 @@ get-deps:
@echo "getting pre go module dependnecies"
go get github.com/jmespath/go-jmespath

get-deps-x-tests:
@echo "go get SDK testing golang.org/x dependencies"
go get golang.org/x/net/http2

get-deps-verify:
@echo "go get SDK verification utilities"
go get golang.org/x/lint/golint
Expand Down
2 changes: 1 addition & 1 deletion example/aws/proxyWithTLSClientCert/main.go
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// +build example
// +build example,go1.15

package main

Expand Down

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions private/model/cli/gen-api/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -293,7 +293,7 @@ func writeAPIErrorsFile(g *generateInfo) error {
func writeAPIEventStreamTestFile(g *generateInfo) error {
return writeGoFile(filepath.Join(g.PackageDir, "eventstream_test.go"),
codeLayout,
"// +build go1.10\n",
"// +build go1.15\n",
g.API.PackageName(),
g.API.APIEventStreamTestGoCode(),
)
Expand All @@ -311,7 +311,7 @@ func writeS3ManagerUploadInputFile(g *generateInfo) error {
func writeAPISmokeTestsFile(g *generateInfo) error {
return writeGoFile(filepath.Join(g.PackageDir, "integ_test.go"),
codeLayout,
"// +build go1.10,integration\n",
"// +build go1.15,integration\n",
g.API.PackageName()+"_test",
g.API.APISmokeTestsGoCode(),
)
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// +build go1.10
// +build go1.15

package eventstreamtest

Expand Down
2 changes: 1 addition & 1 deletion private/protocol/eventstream/eventstreamtest/testing.go
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// +build go1.10
// +build go1.15

package eventstreamtest

Expand Down
2 changes: 1 addition & 1 deletion service/acm/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/apigateway/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/applicationautoscaling/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/applicationdiscoveryservice/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/appstream/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/athena/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/autoscaling/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/batch/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/cloudformation/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/cloudfront/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/cloudhsmv2/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/cloudsearch/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/cloudtrail/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/cloudwatch/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/cloudwatchevents/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/cloudwatchlogs/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/codebuild/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/codecommit/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/codedeploy/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/codepipeline/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/codestar/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/cognitoidentity/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/cognitoidentityprovider/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/cognitosync/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/configservice/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/costandusagereportservice/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/databasemigrationservice/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/devicefarm/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/directconnect/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/directoryservice/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/docdb/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/dynamodb/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/ec2/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/ecr/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/ecs/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/efs/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/elasticache/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion service/elasticbeanstalk/integ_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 66dee9c

Please sign in to comment.