Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Expose OIDC discovery information under the CSAPI #16726

Merged
merged 6 commits into from
Dec 6, 2023

Conversation

DMRobertson
Copy link
Contributor

After this change, we continue to expose discovery information in the client well-known response for backwards compatability.

The MSC 1 includes a new actions_supported which I don't think we currently have. I haven't done anything to add it in. LMK if you'd like me to do so.

David Robertson added 2 commits December 4, 2023 19:14
After this change, we continue to expose discovery information in the
client well-known response for backwards compatability.

The MSC [1] includes a new `actions_supported` which I don't think we
currently have. I haven't done anything to add it in. LMK if you'd like
me to do so.

[1]: matrix-org/matrix-spec-proposals#2965
@DMRobertson DMRobertson marked this pull request as ready for review December 4, 2023 19:18
@DMRobertson DMRobertson requested a review from a team as a code owner December 4, 2023 19:18
Copy link
Member

@sandhose sandhose left a comment

Choose a reason for hiding this comment

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

There is likely to be another iteration on the MSC to get rid of the account URL stuff, but it does not hurt to have that extra info

tests/rest/client/test_auth_issuer.py Outdated Show resolved Hide resolved
tests/rest/client/test_auth_issuer.py Outdated Show resolved Hide resolved
synapse/rest/client/auth_issuer.py Outdated Show resolved Hide resolved
David Robertson and others added 3 commits December 5, 2023 10:13
Co-authored-by: Quentin Gliech <quenting@element.io>
Co-authored-by: Quentin Gliech <quenting@element.io>
@DMRobertson DMRobertson merged commit fcf3c49 into develop Dec 6, 2023
41 checks passed
@DMRobertson DMRobertson deleted the dmr/oidc-discovery branch December 6, 2023 13:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants