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

Upgrade ring to 0.17.5 #3116

Merged
merged 4 commits into from
Oct 31, 2023
Merged

Upgrade ring to 0.17.5 #3116

merged 4 commits into from
Oct 31, 2023

Conversation

jdisanti
Copy link
Collaborator

@jdisanti jdisanti commented Oct 30, 2023

Fixes #3112.

I opted to upgrade to the latest 0.17.5 in spite of the guidance in RFC-21 since this is a security critical dependency, and the maintainer has aggressively yanked old versions in the past.

Note: ring 0.16 is still pulled in by rcgen as a dev dependency in aws-smithy-http-server-python after these changes. The rcgen crate hasn't upgraded yet.


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

@jdisanti jdisanti mentioned this pull request Oct 30, 2023
@github-actions
Copy link

A new generated diff is ready to view.

  • AWS SDK (ignoring whitespace)
  • No codegen difference in the Client Test
  • No codegen difference in the Server Test
  • No codegen difference in the Server Test Python
  • No codegen difference in the Server Test Typescript

A new doc preview is ready to view.

aws/rust-runtime/aws-config/Cargo.toml Outdated Show resolved Hide resolved
aws/rust-runtime/aws-sigv4/Cargo.toml Outdated Show resolved Hide resolved
@jdisanti jdisanti marked this pull request as ready for review October 31, 2023 00:05
@jdisanti jdisanti requested review from a team as code owners October 31, 2023 00:05
@github-actions
Copy link

A new generated diff is ready to view.

  • AWS SDK (ignoring whitespace)
  • No codegen difference in the Client Test
  • No codegen difference in the Server Test
  • No codegen difference in the Server Test Python
  • No codegen difference in the Server Test Typescript

A new doc preview is ready to view.

@jdisanti jdisanti added this pull request to the merge queue Oct 31, 2023
Merged via the queue into main with commit 1ae4d19 Oct 31, 2023
40 of 41 checks passed
@jdisanti jdisanti deleted the jdisanti-upgrade-ring branch October 31, 2023 17:29
rcoh pushed a commit that referenced this pull request Nov 1, 2023
Fixes #3112.

I opted to upgrade to the latest 0.17.5 in spite of the guidance in
[RFC-21](https://github.com/awslabs/smithy-rs/blob/main/design/src/rfcs/rfc0021_dependency_versions.md)
since this is a security critical dependency, and the maintainer has
aggressively yanked old versions in the past.

Note: ring 0.16 is still pulled in by rcgen as a dev dependency in
aws-smithy-http-server-python after these changes. The rcgen crate
hasn't upgraded yet.

----

_By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice._
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.

Update ring to 0.17
2 participants