Skip to content

[Java] Remove AWS SDK v1 STS dependency from serializer-deserializer#503

Open
jvdadda wants to merge 1 commit intoawslabs:masterfrom
mobsuccess-devops:remove-sdk-v1-sts-serializer-deserializer
Open

[Java] Remove AWS SDK v1 STS dependency from serializer-deserializer#503
jvdadda wants to merge 1 commit intoawslabs:masterfrom
mobsuccess-devops:remove-sdk-v1-sts-serializer-deserializer

Conversation

@jvdadda
Copy link
Copy Markdown

@jvdadda jvdadda commented Mar 9, 2026

Issue #, if available: #500

Description of changes:

Remove the com.amazonaws:aws-java-sdk-sts (AWS SDK v1) dependency from the serializer-deserializer module. The v2 equivalent software.amazon.awssdk:sts is already declared and no v1 STS classes are used in source code.

Also updates Javadoc comments referencing the v1 class name AWSCredentialsProvider to the v2 name AwsCredentialsProvider.

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

The v2 STS dependency (software.amazon.awssdk:sts) is already present.
Also update Javadoc references from AWSCredentialsProvider to AwsCredentialsProvider.
Copy link
Copy Markdown

Copilot AI left a comment

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 removes the AWS SDK v1 STS dependency (com.amazonaws:aws-java-sdk-sts) from the serializer-deserializer module, as part of the broader effort to fully remove AWS Java SDK v1 references (issue #500). The v2 equivalent (software.amazon.awssdk:sts) was already declared and no v1 STS classes were used in the source code. Additionally, Javadoc comments referencing the v1 class name AWSCredentialsProvider are updated to the v2 name AwsCredentialsProvider.

Changes:

  • Removed the com.amazonaws:aws-java-sdk-sts dependency from serializer-deserializer/pom.xml
  • Updated Javadoc comments in AWSKafkaAvroDeserializer and GlueSchemaRegistryKafkaDeserializer to reference AwsCredentialsProvider instead of AWSCredentialsProvider
  • Updated corresponding test comments to match the new class name

Reviewed changes

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

Show a summary per file
File Description
serializer-deserializer/pom.xml Removed AWS SDK v1 STS dependency
serializer-deserializer/src/main/java/.../AWSKafkaAvroDeserializer.java Updated Javadoc to reference AwsCredentialsProvider
serializer-deserializer/src/main/java/.../GlueSchemaRegistryKafkaDeserializer.java Updated Javadoc to reference AwsCredentialsProvider
serializer-deserializer/src/test/java/.../AWSKafkaAvroDeserializerTest.java Updated test comment to reference AwsCredentialsProvider
serializer-deserializer/src/test/java/.../GlueSchemaRegistryKafkaDeserializerTest.java Updated test comment to reference AwsCredentialsProvider

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

You can also share your feedback on Copilot code review. Take the survey.

@jvdadda jvdadda changed the title Remove AWS SDK v1 STS dependency from serializer-deserializer [Java] Remove AWS SDK v1 STS dependency from serializer-deserializer Mar 9, 2026
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