Skip to content

Fix streaming signers#2180

Merged
sbiscigl merged 2 commits intomainfrom
fix-streaming-signers
Nov 7, 2022
Merged

Fix streaming signers#2180
sbiscigl merged 2 commits intomainfrom
fix-streaming-signers

Conversation

@sbiscigl
Copy link
Copy Markdown
Collaborator

@sbiscigl sbiscigl commented Nov 7, 2022

Description of changes:

There was a typo in code gen that effected how constructors were created and omitted the stream signer in cases when it was needed. This fixes that and the stream signer will be used again.

Check all that applies:

  • Did a review by yourself.
  • [] Added proper tests to cover this PR. (If tests are not applicable, explain.)
  • Checked if this PR is a breaking (APIs have been changed) change.
  • Checked if this PR will not introduce cross-platform inconsistent behavior.
  • Checked if this PR would require a ReadMe/Wiki update.

Check which platforms you have built SDK on to verify the correctness of this PR.

  • Linux
  • Windows
  • Android
  • MacOS
  • IOS
  • Other Platforms

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@sbiscigl sbiscigl marked this pull request as ready for review November 7, 2022 20:44
@sbiscigl sbiscigl merged commit 021cbf6 into main Nov 7, 2022
@sbiscigl sbiscigl deleted the fix-streaming-signers branch November 7, 2022 20:45
jmklix pushed a commit that referenced this pull request Aug 11, 2023
* fixes stream signing
* run code re-generation
amit-schreiber-firebolt pushed a commit to firebolt-analytics/aws-sdk-cpp that referenced this pull request May 8, 2025
* fixes stream signing
* run code re-generation
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