Skip to content

MSC2965: OAuth 2.0 Authorization Server Metadata discovery #2965

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
turt2live merged 42 commits into from
Mar 29, 2025
Merged

MSC2965: OAuth 2.0 Authorization Server Metadata discovery #2965

turt2live merged 42 commits into from
Mar 29, 2025

Conversation

@sandhose
Copy link
Member

@sandhose sandhose commented Jan 14, 2021 •
edited
Loading

Rendered

Status:

  • Spec is feature complete
  • Reviewed for consistency with MSC3861
  • Implementations believed to be complete enough

Dependencies:

Clients and homeservers currently implement an older version of this proposal, and need to be updated:

SCT:

tickyboxes
checklist

@turt2live turt2live changed the title MSC2965: [WIP] OIDC Provider discovery [WIP] MSC2965: OIDC Provider discovery Jan 14, 2021
@turt2live turt2live marked this pull request as draft January 14, 2021 17:27
@turt2live turt2live added kind:feature MSC for not-core and not-maintenance stuff proposal A matrix spec change proposal labels Jan 14, 2021
@sandhose sandhose mentioned this pull request Jan 15, 2021
Merged
9 tasks
@turt2live turt2live added the needs-implementation This MSC does not have a qualifying implementation for the SCT to review. The MSC cannot enter FCP. label Jun 8, 2021
This was referenced Jun 22, 2022
Merged
Closed
@hughns hughns mentioned this pull request Aug 4, 2022
Merged
@erlend-sh
Copy link

erlend-sh commented Aug 9, 2022 •
edited
Loading

Keycloak in OIDC Playground

Are any other examples planned?

I’m using Ory for several apps that I’d like to also connect together with Matrix. It also strikes me as a conveniently lightweight example for Matrix, which also aligns well with Dendrite since it’s in Go.

@hughns hughns mentioned this pull request Aug 14, 2022
Open
@hughns
Copy link
Member

hughns commented Aug 14, 2022

@erlend-sh Good suggestion, thank you - I've added element-hq/oidc-playground#3 to track this.

@hughns hughns changed the title [WIP] MSC2965: OIDC Provider discovery MSC2965: OIDC Provider discovery Sep 22, 2022
@hughns hughns marked this pull request as ready for review September 22, 2022 16:08
@hughns hughns mentioned this pull request Jan 9, 2023
Merged
20 tasks
@turt2live turt2live added the matrix-2.0 Required for Matrix 2.0 label Mar 3, 2023
clokep
clokep reviewed May 16, 2023
View reviewed changes
clokep
clokep reviewed May 16, 2023
View reviewed changes
clokep
clokep reviewed May 16, 2023
View reviewed changes
clokep
clokep reviewed May 16, 2023
View reviewed changes
clokep
clokep reviewed May 16, 2023
View reviewed changes
clokep
clokep reviewed May 16, 2023
View reviewed changes
clokep
clokep reviewed May 16, 2023
View reviewed changes
@mscbot
Copy link
Collaborator

mscbot commented Mar 24, 2025

🔔 This is now entering its final comment period, as per the review above. 🔔

@mscbot mscbot added final-comment-period This MSC has entered a final comment period in interest to approval, postpone, or delete in 5 days. and removed proposed-final-comment-period Currently awaiting signoff of a majority of team members in order to enter the final comment period. labels Mar 24, 2025
@turt2live turt2live moved this from Ready for FCP ticks to In FCP in Spec Core Team Workflow Mar 25, 2025
@mscbot
Copy link
Collaborator

mscbot commented Mar 29, 2025

The final comment period, with a disposition to merge, as per the review above, is now complete.

@mscbot mscbot added finished-final-comment-period and removed disposition-merge final-comment-period This MSC has entered a final comment period in interest to approval, postpone, or delete in 5 days. labels Mar 29, 2025
@turt2live turt2live merged commit 1e689aa into matrix-org:main Mar 29, 2025
1 check passed
@turt2live turt2live moved this from In FCP to Requires spec writing in Spec Core Team Workflow Mar 29, 2025
@turt2live turt2live added spec-pr-missing Proposal has been implemented and is being used in the wild but hasn't yet been added to the spec and removed finished-final-comment-period labels Mar 29, 2025
yostyle pushed a commit to tchapgouv/matrix-rust-sdk that referenced this pull request Apr 4, 2025
@zecakeh
refactor(oidc): Use the GET /auth_metadata Matrix endpoint (matrix-…
c2a2691 
…org#4673)

This is the method to get the server metadata in the latest draft of
[MSC2965](matrix-org/matrix-spec-proposals#2965).

We still keep the old behavior with `GET /auth_issuer` as fallback for
now because it has wider server support.

There are some pre-main commit cleanups to simplify the main commit.
This can be reviewed commit by commit.

The changes were tested with the oidc_cli example on beta.matrix.org.

Closes matrix-org#4550.

---------

Signed-off-by: Kévin Commaille <[email protected]>
@NinekoTheCat NinekoTheCat mentioned this pull request Apr 18, 2025
Open
@zecakeh
Copy link
Contributor

zecakeh commented May 24, 2025

spec PR: matrix-org/matrix-spec#2147

@tulir tulir added spec-pr-in-review A proposal which has been PR'd against the spec and is in review and removed spec-pr-missing Proposal has been implemented and is being used in the wild but hasn't yet been added to the spec labels May 24, 2025
@HarHarLinks HarHarLinks mentioned this pull request May 26, 2025
Merged
@turt2live turt2live moved this from Requires spec writing to Requires spec PR review in Spec Core Team Workflow Jun 10, 2025
@richvdh
Copy link
Member

richvdh commented Jun 17, 2025

spec PR: matrix-org/matrix-spec#2147

Merged!

@richvdh richvdh added merged A proposal whose PR has merged into the spec! and removed spec-pr-in-review A proposal which has been PR'd against the spec and is in review labels Jun 17, 2025
@turt2live turt2live moved this from Requires spec PR review to Done to some definition in Spec Core Team Workflow Jun 17, 2025
@zecakeh zecakeh mentioned this pull request Jun 21, 2025
Closed
35 tasks
@Johennes Johennes mentioned this pull request Jul 3, 2025
Closed
34 tasks
@turt2live turt2live mentioned this pull request Jul 24, 2025
Open
34 tasks
noelportillo

This comment was marked as spam.

Sign in to view

Copilot AI mentioned this pull request Oct 13, 2025
Draft
@krille-chan krille-chan mentioned this pull request Nov 4, 2025
Open
1 task
@chbgdn chbgdn mentioned this pull request Jan 15, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@andybalaam andybalaam andybalaam left review comments

@clokep clokep clokep left review comments

@uhoreg uhoreg uhoreg left review comments

@richvdh richvdh richvdh left review comments

@tulir tulir tulir left review comments

@turt2live turt2live turt2live approved these changes

@erikjohnston erikjohnston Awaiting requested review from erikjohnston

+5 more reviewers

@Johennes Johennes Johennes left review comments

@oliverpool oliverpool oliverpool left review comments

@tonkku107 tonkku107 tonkku107 left review comments

@zecakeh zecakeh zecakeh left review comments

@noelportillo noelportillo noelportillo left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

kind:core MSC which is critical to the protocol's success matrix-2.0 Required for Matrix 2.0 merged A proposal whose PR has merged into the spec! proposal A matrix spec change proposal

Projects

Spec Core Team Workflow
Status: Merged

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.