Skip to content

Fixes #3470 - #3471

Merged
Jean-Marc Prieur (jmprieur) merged 4 commits into
masterfrom
jmprieur/Fix3470
Sep 10, 2025
Merged

Jean-Marc Prieur (jmprieur) merged 4 commits into
masterfrom
jmprieur/Fix3470

Conversation

@jmprieur

Copy link
Copy Markdown
Collaborator

Fixes #3470

Description

  • Remove the client secret (whatever its value) when there is a client assertion
  • Don't retry when the client secret is wrong (discovered when testing)

Tested manually with a client secret temporarily added to the app.
Not providing an automated test because of SFI

Fixes #3470

Tested manually with a client secret.
@jmprieur
Jean-Marc Prieur (jmprieur) requested a review from a team as a code owner September 9, 2025 20:38
@jmprieur
Jean-Marc Prieur (jmprieur) deleted the jmprieur/Fix3470 branch September 10, 2025 01:38
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.

Acquiring tokens for Agent identities should be possible with Client secrets

4 participants