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

aadV2Token sample in readme shows clientid (all lowercase), but it should be clientId #1348

Open
fpintos opened this issue Jan 29, 2025 · 0 comments

Comments

@fpintos
Copy link

fpintos commented Jan 29, 2025

I tried to follow the instructions for setting up $aadV2Token and used tenantid and clientid as per sample in the readme,

* `{{$aadV2Token [new] [AzureCloud|AzureChinaCloud|AzureUSGovernment|ppe] [appOnly ][scopes:<scope[,]>] [tenantid:<domain|tenantId>] [clientid:<clientId>]}}`: Add an Azure Active Directory token based on the following options (must be specified in order):

It kept failing sending the default client ID, and eventually I noticed the case mismatch.
In src/utils/aadV2TokenProvider.ts, the regex aadV2TokenRegex asks for tenantId and clientId.

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

No branches or pull requests

1 participant