Skip to content

Switch OIDC to response_mode=fragment#33100

Merged
t3chguy merged 3 commits intodevelopfrom
t3chguy/oidc-fragment
Apr 15, 2026
Merged

Switch OIDC to response_mode=fragment#33100
t3chguy merged 3 commits intodevelopfrom
t3chguy/oidc-fragment

Conversation

@t3chguy
Copy link
Copy Markdown
Member

@t3chguy t3chguy commented Apr 10, 2026

To avoid the code & state being visible to the web server

Review commit-by-commit, starts with a refactor

Requires matrix-org/matrix-js-sdk#5265
Requires matrix-org/matrix-js-sdk#5266

@t3chguy t3chguy self-assigned this Apr 10, 2026
@t3chguy t3chguy force-pushed the t3chguy/oidc-fragment branch from 57d1d1d to daafaa2 Compare April 10, 2026 13:47
@t3chguy t3chguy force-pushed the t3chguy/oidc-fragment branch from 8223797 to aa20ae1 Compare April 13, 2026 13:50
@t3chguy t3chguy force-pushed the t3chguy/oidc-fragment branch from aa20ae1 to a302cc5 Compare April 13, 2026 14:28
@t3chguy t3chguy force-pushed the t3chguy/oidc-fragment branch from a302cc5 to 4146292 Compare April 13, 2026 14:54
@t3chguy t3chguy force-pushed the t3chguy/oidc-fragment branch from 49145d7 to 7c5050d Compare April 14, 2026 08:35
@t3chguy t3chguy force-pushed the t3chguy/oidc-fragment branch from 7c5050d to ae24f4a Compare April 14, 2026 10:27
@t3chguy

This comment was marked as outdated.

@t3chguy t3chguy marked this pull request as ready for review April 14, 2026 15:21
@t3chguy t3chguy requested a review from a team as a code owner April 14, 2026 15:21
@t3chguy t3chguy requested review from Half-Shot and dbkr April 14, 2026 15:21
@t3chguy t3chguy added this pull request to the merge queue Apr 15, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 15, 2026
@t3chguy t3chguy added this pull request to the merge queue Apr 15, 2026
Merged via the queue into develop with commit de4a1e6 Apr 15, 2026
94 of 97 checks passed
@t3chguy t3chguy deleted the t3chguy/oidc-fragment branch April 15, 2026 10:07
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.

2 participants