Skip to content

deprecate(Q): remove sigv4 sdk #5332

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

Merged
merged 17 commits into from
Mar 13, 2025
Merged

deprecate(Q): remove sigv4 sdk #5332

merged 17 commits into from
Mar 13, 2025

Conversation

Will-ShaoHua
Copy link
Contributor

@Will-ShaoHua Will-ShaoHua commented Feb 6, 2025

Description

Remove unused sigv4 client and mapping back and forward between 2 sdks POJOs

License

I confirm that my contribution is made under the terms of the Apache 2.0 license.

@Will-ShaoHua Will-ShaoHua changed the title remove sigv4 version cwsprClient code path refactor(Q: remove sigv4 sdk Feb 12, 2025
@Will-ShaoHua Will-ShaoHua changed the title refactor(Q: remove sigv4 sdk refactor(Q): remove sigv4 sdk Feb 12, 2025
@Will-ShaoHua Will-ShaoHua requested a review from rli February 12, 2025 18:19
@Will-ShaoHua Will-ShaoHua marked this pull request as ready for review February 12, 2025 18:19
@Will-ShaoHua Will-ShaoHua requested a review from a team as a code owner February 12, 2025 18:19
@Will-ShaoHua
Copy link
Contributor Author

not needed for 2/13 release

@Will-ShaoHua Will-ShaoHua requested a review from ctlai95 February 12, 2025 18:20
@Will-ShaoHua Will-ShaoHua changed the title refactor(Q): remove sigv4 sdk deprecate(Q): remove sigv4 sdk Feb 14, 2025
@Will-ShaoHua Will-ShaoHua marked this pull request as draft February 14, 2025 23:14
@Will-ShaoHua Will-ShaoHua marked this pull request as ready for review February 17, 2025 19:26
@rli rli merged commit cb6daca into aws:main Mar 13, 2025
11 of 14 checks passed
leigaol pushed a commit to leigaol/aws-toolkit-jetbrains that referenced this pull request Mar 20, 2025
Remove unused sigv4 client and mapping back and forward between 2 sdks POJOs
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.

4 participants