Skip to content

fix(oauth2): Add missing urlencode for failure redirection#59757

Merged
Altahrim merged 1 commit intomasterfrom
fix/fix-oauth2-failure-redirect
Apr 21, 2026
Merged

fix(oauth2): Add missing urlencode for failure redirection#59757
Altahrim merged 1 commit intomasterfrom
fix/fix-oauth2-failure-redirect

Conversation

@come-nc
Copy link
Copy Markdown
Contributor

@come-nc come-nc commented Apr 21, 2026

Summary

Fixes a small oversight in oauth2 controller.

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc come-nc added this to the Nextcloud 34 milestone Apr 21, 2026
@come-nc come-nc self-assigned this Apr 21, 2026
@come-nc come-nc requested a review from a team as a code owner April 21, 2026 07:37
@come-nc come-nc requested review from artonge, icewind1991, leftybournes and salmart-dev and removed request for a team April 21, 2026 07:37
@come-nc come-nc added the 3. to review Waiting for reviews label Apr 21, 2026
@come-nc
Copy link
Copy Markdown
Contributor Author

come-nc commented Apr 21, 2026

/backport to stable33

@come-nc
Copy link
Copy Markdown
Contributor Author

come-nc commented Apr 21, 2026

/backport to stable32

@come-nc
Copy link
Copy Markdown
Contributor Author

come-nc commented Apr 21, 2026

/backport to stable31

@come-nc come-nc enabled auto-merge April 21, 2026 08:22
@Altahrim Altahrim disabled auto-merge April 21, 2026 09:04
@Altahrim Altahrim merged commit 94f1446 into master Apr 21, 2026
189 of 194 checks passed
@Altahrim Altahrim deleted the fix/fix-oauth2-failure-redirect branch April 21, 2026 09:04
@come-nc
Copy link
Copy Markdown
Contributor Author

come-nc commented Apr 21, 2026

/backport to stable30

@come-nc
Copy link
Copy Markdown
Contributor Author

come-nc commented Apr 21, 2026

/backport to stable29

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

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants