Skip to content

fix: revoke by consent request ID#3947

Merged
alnr merged 1 commit intomasterfrom
alnr/revoke-consent
Feb 20, 2025
Merged

fix: revoke by consent request ID#3947
alnr merged 1 commit intomasterfrom
alnr/revoke-consent

Conversation

@alnr
Copy link
Copy Markdown
Contributor

@alnr alnr commented Feb 18, 2025

This is a follow-up to #3932.

This makes several improvements, chiefly restoring backwards compatibility to Hydra before #3932. We now return both the challenge as well as the consent_request_id from our APIs and distinguish between them clearly.

Closes #3941

@alnr alnr self-assigned this Feb 18, 2025
@alnr alnr requested review from a team and aeneasr as code owners February 18, 2025 16:27
@alnr alnr force-pushed the alnr/revoke-consent branch from ea6b99d to b747943 Compare February 19, 2025 17:28
Copy link
Copy Markdown
Contributor

@hperl hperl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@alnr alnr added this pull request to the merge queue Feb 20, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Feb 20, 2025
@alnr alnr merged commit 5d8635c into master Feb 20, 2025
29 checks passed
@alnr alnr deleted the alnr/revoke-consent branch February 20, 2025 18:41
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.

Follow-ups to #3932

2 participants