Skip to content

Switch AWS SDK to default-https-client - #5

Merged
yuhao-su merged 3 commits into
yuhao-su:mainfrom
igorscs:switch-to-new-rustls
Jan 30, 2026
Merged

yuhao-su merged 3 commits into
yuhao-su:mainfrom
igorscs:switch-to-new-rustls

Conversation

@igorscs

@igorscs igorscs commented Jan 23, 2026

Copy link
Copy Markdown
Contributor

Avoid legacy rustls feature in aws-config/aws-sdk-sts as it brings legacy rustls 0.21
See note about changes to the default HTTPS client stack awslabs/aws-sdk-rust#1257

@igorscs

igorscs commented Jan 28, 2026 •

Copy link
Copy Markdown
Contributor Author

if you decide to merge, would you please make a new release?

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR switches AWS SDK dependencies from the legacy rustls feature to default-https-client, avoiding the inclusion of legacy rustls 0.21 in the dependency tree. The change aligns with AWS SDK recommendations for the default HTTPS client stack.

Changes:

  • Updated aws-config and aws-sdk-sts feature flags from rustls to default-https-client in Cargo.toml
  • Updated Cargo.lock with newer dependency versions including modern rustls 0.23.31 and aws-lc-rs cryptographic provider

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
Cargo.toml Replaced rustls feature with default-https-client for aws-config and aws-sdk-sts dependencies
Cargo.lock Updated dependency tree to use modern rustls 0.23.31, added aws-smithy-http-client and aws-lc-rs, removed legacy rustls 0.21 dependencies

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Igor Smolyar and others added 2 commits January 30, 2026 03:36
Avoid legacy rustls feature in aws-config/aws-sdk-sts
as it brings legacy rustls 0.21
See note about changes to the default HTTPS client stack
awslabs/aws-sdk-rust#1257
@yuhao-su
yuhao-su force-pushed the switch-to-new-rustls branch from f222939 to 8a34f22 Compare January 30, 2026 09:46
@yuhao-su
yuhao-su merged commit c46f5c2 into yuhao-su:main Jan 30, 2026
7 checks passed
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