Skip to content

Use SORT_STRING for canonicalized query generation#3261

Merged
stobrien89 merged 1 commit into
aws:masterfrom
simonkey:fix-signature-v4-canonicalized-query
Mar 17, 2026
Merged

Use SORT_STRING for canonicalized query generation#3261
stobrien89 merged 1 commit into
aws:masterfrom
simonkey:fix-signature-v4-canonicalized-query

Conversation

@simonkey

Copy link
Copy Markdown
Contributor

Issue #, if available: #3258

Description of changes:
Use SORT_STRING flag for sorting. Added a test for the canonicalized query generation.

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

@yenfryherrerafeliz

Copy link
Copy Markdown
Contributor
vendor/bin/behat --format=progress --suite=smoke --tags='~@noassumerole'
...................................................................... 70
...................................................................... 140
...................................................................... 210
............

111 scenarios (111 passed)
222 steps (222 passed)
0m36.14s (94.00Mb)
vendor/bin/behat --format=progress --tags=integ
...................................................................... 70
...................................................................... 140
...................................................................... 210
...................................................................... 280
...................................................................... 350
...................................................................... 420
...............

127 scenarios (127 passed)
435 steps (435 passed)
21m17.00s (45.63Mb)

@stobrien89 stobrien89 force-pushed the fix-signature-v4-canonicalized-query branch from 685bd42 to f1ae2ae Compare March 17, 2026 17:48
@stobrien89 stobrien89 merged commit 61c73ef into aws:master Mar 17, 2026
17 checks passed
@stobrien89

Copy link
Copy Markdown
Member

Thanks for the contribution @simonkey!

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