Skip to content

Add support for DefaultAzureCredentials #14

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

Merged
merged 6 commits into from
Mar 10, 2025
Merged

Conversation

atakavci
Copy link
Collaborator

Azure offers a streamlined authentication solution for TBA scenarios through a unified construct called DefaultAzureCredentials. This approach consolidates multiple credential configurations commonly used in Azure-hosted environments, simplifying authentication management.
This pull request introduces support for DefaultAzureCredentials within our extension library.
More details on DefaultAzureCredentials;
https://learn.microsoft.com/en-us/java/api/com.azure.identity.defaultazurecredential?view=azure-java-stable
https://learn.microsoft.com/en-us/java/api/overview/azure/identity-readme?view=azure-java-stable

@atakavci atakavci added the enhancement New feature or request label Feb 25, 2025
@atakavci atakavci requested review from uglide, tishun and ggivo February 25, 2025 13:34
@atakavci atakavci self-assigned this Feb 25, 2025
Copy link

@tishun tishun left a comment

Choose a reason for hiding this comment

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

Do we want to add some more JavaDoc?

Copy link
Contributor

@ggivo ggivo left a comment

Choose a reason for hiding this comment

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

LGTM

@ggivo
Copy link
Contributor

ggivo commented Mar 7, 2025

Do we want to add some more JavaDoc?
+1

@atakavci
Copy link
Collaborator Author

@tishun @ggivo thanks guys, added some javadoc. i ll go for formatting for all in another round.

@atakavci atakavci merged commit 077e460 into main Mar 10, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants