Skip to content

fix: remove AWS SDK v1 dependencies and use AWS SDK v2 in neptune backend#2913

Merged
kodiakhq[bot] merged 2 commits intoguacsec:mainfrom
gaganhr94:fix/move-aws-v1-to-v2
Mar 31, 2026
Merged

fix: remove AWS SDK v1 dependencies and use AWS SDK v2 in neptune backend#2913
kodiakhq[bot] merged 2 commits intoguacsec:mainfrom
gaganhr94:fix/move-aws-v1-to-v2

Conversation

@gaganhr94
Copy link
Copy Markdown
Contributor

Description of the PR

Replace github.com/aws/aws-sdk-go (v1) with github.com/aws/aws-sdk-go-v2 in the Neptune backend. The v1 SDK is deprecated and no longer receiving updates. The v2 equivalents for session, credentials, and signer/v4 are used, preserving the existing credential resolution logic (env vars first, then default config fallback)

Fixes #2912

PR Checklist

  • All commits have a Developer Certificate of Origin (DCO) -- they are generated using -s flag to git commit.
  • All new changes are covered by tests
  • If GraphQL schema is changed, make generate has been run
  • If GraphQL schema is changed, GraphQL client updates/additions have been made
  • If OpenAPI spec is changed, make generate has been run
  • If ent schema is changed, make generate has been run
  • If collectsub protobuf has been changed, make proto has been run
  • All CI checks are passing (tests and formatting)
  • All dependent PRs have already been merged

…kend

Signed-off-by: Gagan H R <hrgagan4@gmail.com>
@gaganhr94 gaganhr94 force-pushed the fix/move-aws-v1-to-v2 branch from aac7915 to f44ff24 Compare March 15, 2026 12:17
Copy link
Copy Markdown
Member

@mihaimaruseac mihaimaruseac left a comment

Choose a reason for hiding this comment

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

Thank you!

@mihaimaruseac
Copy link
Copy Markdown
Member

We also need to resolve conflicts here

@gaganhr94
Copy link
Copy Markdown
Contributor Author

Done @mihaimaruseac, it is rebased with main now

@mihaimaruseac
Copy link
Copy Markdown
Member

Awesome. We need one more reviewer and then it should merge

@kodiakhq kodiakhq bot merged commit d7ecdfc into guacsec:main Mar 31, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate backend from deprecated aws-sdk-go (v1) to aws-sdk-go-v2

3 participants