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

Fix kms #68

Merged
merged 4 commits into from
Oct 4, 2023
Merged

Fix kms #68

merged 4 commits into from
Oct 4, 2023

Conversation

petreeftime
Copy link
Contributor

Issue #, if available:

Description of changes:

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

aws_config and aws_sdk_kms no longer matched, causing for this feature
to not compile. This patch bring the aws_sdk_kms to latest version,
which works with the latest aws_config.

Signed-off-by: Petre Eftime <petre.eftime@gmail.com>
Unfortunately tests require AWS creds, so skip running the tests, but at
least make sure that the feature still compiles correctly.

Signed-off-by: Petre Eftime <petre.eftime@gmail.com>
AWS SDK and time crate no longer compile with 1.58 and need minimum
1.67.

Signed-off-by: Petre Eftime <petre.eftime@gmail.com>
Fix warnings about deprecated node versions for checkout v2.

Signed-off-by: Petre Eftime <petre.eftime@gmail.com>
Copy link
Contributor

@belveryin belveryin left a comment

Choose a reason for hiding this comment

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

CI / Test on rust 1.66 (keys key_kms) (pull_request) doesn't pass, it requires 1.67 version of rust.

@petreeftime
Copy link
Contributor Author

Fixed, and also included a fix for the warnings regarding node12 deprecation for actions/checkout@v2.

@belveryin belveryin merged commit b0950d2 into awslabs:main Oct 4, 2023
9 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.

None yet

2 participants