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

Compatibility with AWS-CDK >= 2.167.0 #96

Merged
merged 4 commits into from
Nov 20, 2024
Merged

Compatibility with AWS-CDK >= 2.167.0 #96

merged 4 commits into from
Nov 20, 2024

Conversation

dominikschubert
Copy link
Member

@dominikschubert dominikschubert commented Nov 20, 2024

Context

With https://github.com/aws/aws-cdk/releases/tag/v2.167.0 aws-cdk switched to the JavaScript AWS SDK v3 which has a completely different way of instantiating and configuring clients. This unfortunately broke the integration with LocalStack as the clients were using their default endpoints (i.e. targeting AWS).

Addresses #95

Note: Maintainence here is really getting a bit hard with all the different levels of undefined compatibility for different versions. A lot of the code is completely unnecessary at this point for the latest aws-cdk versions.

Would love to discuss this in the future, but for now this is a quick and easy fix to patch the correct endpoints into the clients/configs.

@dominikschubert dominikschubert marked this pull request as ready for review November 20, 2024 10:48
Copy link

@simonrw simonrw left a comment

Choose a reason for hiding this comment

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

Thanks for implementing this and fixing cdklocal for new aws-cdk-lib versions.

Can we update the changelog in the README.md (could be in a follow up) to describe this fix?

@dominikschubert dominikschubert merged commit bee3198 into main Nov 20, 2024
12 checks passed
@dominikschubert dominikschubert deleted the js-sdk-v3 branch November 20, 2024 12:20
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.

2 participants