Skip to content

Add Azure Application Credential - #23178

Merged
g2vinay merged 6 commits into
Azure:mainfrom
g2vinay:add-azure-application-credential
Aug 10, 2021
Merged

Add Azure Application Credential#23178
g2vinay merged 6 commits into
Azure:mainfrom
g2vinay:add-azure-application-credential

Conversation

@g2vinay

@g2vinay g2vinay commented Jul 26, 2021

Copy link
Copy Markdown
Member

Fixes #22366

@ghost ghost added the Azure.Identity label Jul 26, 2021
@Immutable
public final class AzureApplicationCredential extends ChainedTokenCredential {
/**
* Creates default AzureApplicationCredential instance to use. This will use environment variables to create

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

remove default

*
* @see AzureApplicationCredential
*/
public class AzureApplicationCredentialBuilder extends CredentialBuilderBase<AzureApplicationCredentialBuilder> {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Make this final

@g2vinay

g2vinay commented Aug 10, 2021

Copy link
Copy Markdown
Member Author

/check-enforcer evaluate

@g2vinay
g2vinay marked this pull request as ready for review August 10, 2021 16:27
@g2vinay

g2vinay commented Aug 10, 2021

Copy link
Copy Markdown
Member Author

/check-enforcer override

@g2vinay g2vinay closed this Aug 10, 2021
@g2vinay g2vinay reopened this Aug 10, 2021
@g2vinay

g2vinay commented Aug 10, 2021

Copy link
Copy Markdown
Member Author

/check-enforcer evaluate

@g2vinay

g2vinay commented Aug 10, 2021

Copy link
Copy Markdown
Member Author

/check - enforcer override

@g2vinay

g2vinay commented Aug 10, 2021

Copy link
Copy Markdown
Member Author

/check-enforcer evaluate

@g2vinay
g2vinay merged commit 7511d8a into Azure:main Aug 10, 2021
@g2vinay
g2vinay deleted the add-azure-application-credential branch July 13, 2023 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create AzureApplicationCredential

2 participants