8000 Ensure consent is granted before token-exchange by cavuugroup · Pull Request #26908 · keycloak/keycloak · GitHub
[go: up one dir, main page]

Skip to content

Ensure consent is granted before token-exchange - #26908

Closed
cavuugroup wants to merge 4 commits into
keycloak:mainfrom
cavuugroup:issue/KC/20702
Closed

cavuugroup wants to merge 4 commits into
keycloak:mainfrom
cavuugroup:issue/KC/20702

Conversation

@cavuugroup
@cavuugroup cavuugroup commented Feb 8, 2024 •
Copy link
Copy Markdown

Closes #20702

Verification of user consents to scopes requested by the target client during client to client token exchange.

Closes keycloak#20702

Signed-off-by: Krzysztof Walczewski <kwalczewski@gmail.com>
@thomasdarimont
Copy link
Copy Markdown
Contributor

@cavuugroup thanks for your contribution! Would you mind changing the title of this PR to:
"Ensure consent is granted before token-exchange"

@ahus1 ahus1 changed the title Chacking consents while token exchanging Ensure consent is granted before token-exchange Jun 26, 2024
@ahus1
ahus1 commented Jun 26, 2024
Copy link
Copy Markdown
Member

@thomasdarimont - I helped out here a bit by changing it as I have write permissions on it. I also see there is a conflict for this PR which needs to be resolved before it can be merged.

@ahus1
ahus1 commented Jul 1, 2024
Copy link
Copy Markdown
Member

@pedroigor / @thomasdarimont - The build is green, and the conflicts are gone. Are those changes good to be merged?

@ahus1 ahus1 assigned pedroigor and unassigned ahus1 Aug 6, 2024
@ofairfoul
Copy link
Copy Markdown

Any chance of this being merged?

@pedroigor pedroigor removed their assignment Aug 12, 2025
@stianst
stianst commented Apr 30, 2026
Copy link
Copy Markdown
Contributor

Since this is for the now deprecated token exchange v1, this should probably be closed. @mposolda wdyt?

@mposolda
Copy link
Copy Markdown
Contributor

Closing as this is for token-exchange V1. The Token-exchange:v2 should work in a way proposed in a PR and in the GH issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Token Exchange should allow exchange for clients that require consent if user has given consent for requested scopes

7 participants

0