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

CosmosClientBuilder: Adds overload for passing TokenCredential #2753

Merged
merged 5 commits into from
Sep 28, 2021
Merged

CosmosClientBuilder: Adds overload for passing TokenCredential #2753

merged 5 commits into from
Sep 28, 2021

Conversation

levimatheri
Copy link
Contributor

@levimatheri levimatheri commented Sep 26, 2021

Description

Adds constructor to CosmosClientBuilder for using a TokenCredential instance, e.g. DefaultAzureCredential.

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update (addition)

Closing issues

Closes #2749

@ghost
Copy link

ghost commented Sep 26, 2021

CLA assistant check
All CLA requirements met.

@ealsur
Copy link
Member

ealsur commented Sep 27, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@ealsur
Copy link
Member

ealsur commented Sep 27, 2021

@levimatheri please run https://github.com/Azure/azure-cosmos-dotnet-v3/blob/master/UpdateContracts.ps1 on your branch to generate the changes in the API surface. You can execute it in a PS shell at the root of the repo's clone and it should update a JSON file that exists, which you can include in the PR.

@ealsur ealsur changed the title For #2749. Add CosmosClientBuilder overload for passing TokenCredential CosmosClientBuilder: Adds overload for passing TokenCredential Sep 27, 2021
@j82w
Copy link
Contributor

j82w commented Sep 28, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@j82w j82w merged commit 48e1a6b into Azure:master Sep 28, 2021
@ealsur
Copy link
Member

ealsur commented Sep 28, 2021

Thanks @levimatheri for your contribution!

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.

Add TokenCredential builder method to CosmosClientBuilder
5 participants