Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add setting content-length middleware into operation stack #634

Merged
merged 2 commits into from
Jul 15, 2020

Conversation

jasdel
Copy link
Contributor

@jasdel jasdel commented Jul 9, 2020

Adds the ContentLengthMiddleware to the operation's stack, and updates generated clients and protocol tests to use the content-length check.

Depends on:

@jasdel jasdel force-pushed the AddContentLengthMiddleware branch from ec3cda8 to 162e95a Compare July 13, 2020 22:56
Adds the ContentLengthMiddleware to the operation's stack, and updates
generated clients and protocol tests to use the content-length check.

Depends on:
* aws/smithy-go#108
* smithy-lang/smithy#491
@jasdel jasdel force-pushed the AddContentLengthMiddleware branch 2 times, most recently from a583d34 to 2869776 Compare July 14, 2020 23:44
@jasdel jasdel force-pushed the AddContentLengthMiddleware branch from 2869776 to 2f9a05e Compare July 15, 2020 18:40
@jasdel jasdel merged commit e21158e into aws:feature/SmithyAPIClient Jul 15, 2020
@jasdel jasdel deleted the AddContentLengthMiddleware branch July 15, 2020 19:47
skmcgrail pushed a commit to skmcgrail/aws-sdk-go-v2 that referenced this pull request Aug 21, 2020
Adds the ContentLengthMiddleware to the operation's stack, and updates generated clients and protocol tests to use the content-length check.

Depends on:
* aws/smithy-go#108
* smithy-lang/smithy#491
skmcgrail pushed a commit that referenced this pull request Sep 24, 2020
Adds the ContentLengthMiddleware to the operation's stack, and updates generated clients and protocol tests to use the content-length check.

Depends on:
* aws/smithy-go#108
* smithy-lang/smithy#491
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants