-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[Identity] Support exchanging k8s token to AAD token #16688
Changes from all commits
9d396d3
684c32e
5dede8c
63c2986
8f93b29
e376252
d35daf5
18ee518
f8deac2
a19e628
3df153d
ad0f06f
67f1d04
33cd48c
7aea1ee
2dd335c
ab0c572
b87127f
2881f6c
9eb6203
385f187
02fd239
0ec0125
9b3811b
2f07e3a
3091b74
1dd2856
a2a5f89
1cf0242
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -164,7 +164,7 @@ export class AggregateAuthenticationError extends Error { | |
|
||
constructor(errors: any[], errorMessage?: string) { | ||
const errorDetail = errors.join("\n"); | ||
super(`${errorMessage}\n\n${errorDetail}`); | ||
super(`${errorMessage}\n${errorDetail}`); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This meant a whole empty line, which I didn’t like now that I re-reviewed these errors. |
||
this.errors = errors; | ||
|
||
// Ensure that this type reports the correct name | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -80,7 +80,10 @@ export class ChainedTokenCredential implements TokenCredential { | |
} | ||
|
||
if (!token && errors.length > 0) { | ||
const err = new AggregateAuthenticationError(errors); | ||
const err = new AggregateAuthenticationError( | ||
errors, | ||
"ChainedTokenCredential authentication failed." | ||
); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Without this, this error had an initial segment with an |
||
span.setStatus({ | ||
code: SpanStatusCode.ERROR, | ||
message: err.message | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is intentionally vague.