Skip to content

Fix ID token credential type casing - #657

Merged
Nilesh Choudhary (4gust) merged 1 commit into
mainfrom
4gust-fix-cache-key-compatibility
Aug 21, 2026
Merged

Nilesh Choudhary (4gust) merged 1 commit into
mainfrom
4gust-fix-cache-key-compatibility

Conversation

@4gust

Copy link
Copy Markdown
Collaborator

Change the serialized ID-token credential type to IdToken for cache compatibility.\n\nFixes #652

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: fa13b562-583e-4255-90a6-4e30c607f64c
@sonarqubecloud

Copy link
Copy Markdown

@4gust
Nilesh Choudhary (4gust) merged commit 158f372 into main Aug 21, 2026
15 checks passed
@4gust Nilesh Choudhary (4gust) linked an issue Aug 25, 2026 that may be closed by this pull request
1 of 8 tasks
Robbie-Microsoft added a commit that referenced this pull request Aug 27, 2026
Brings the branch up to date with main through 247817f (Prepare 1.9.0
release, #658), including the WS-Trust endpoint selection helper (#655),
the ID token credential type casing fix (#657) and the Service Fabric TLS
certificate pin (#656).

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Robbie-Microsoft added a commit that referenced this pull request Aug 27, 2026
Cascade the updated base branch into the bearer-over-mTLS work. Brings in:

- main merged into the base: 1.9.0 release (#658), WS-Trust endpoint
  selection helper fix (#655), ID token credential type casing (#657),
  and the Service Fabric TLS certificate pin (#656).
- A lint fix adding //nolint:staticcheck at the six sites where the mTLS
  guard must read the deprecated http.Transport.DialTLS hook, clearing
  the staticcheck SA1019 failures in the lint check.
- A comment-only clarification in comm/mtls.go explaining why the DialTLS
  check reads a deprecated field.

No conflicts; no overlap with this branch's 13 changed files. No function
signatures changed, so no call sites needed updating.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
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.

[Bug] Cache keys don't match those from Python MSAL

3 participants