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

workers.md gives incorrect advice about various endpoints #8966

Closed
richvdh opened this issue Dec 17, 2020 · 1 comment · Fixed by #9271
Closed

workers.md gives incorrect advice about various endpoints #8966

richvdh opened this issue Dec 17, 2020 · 1 comment · Fixed by #9271
Labels
A-Docs things relating to the documentation

Comments

@richvdh
Copy link
Member

richvdh commented Dec 17, 2020

https://github.com/matrix-org/synapse/blob/develop/docs/workers.md#synapseappgeneric_worker says:

Additionally, the following endpoints should be included if Synapse is configured to use SSO...

# OpenID Connect requests.
^/_matrix/client/(api/v1|r0|unstable)/login/sso/redirect$
^/_synapse/oidc/callback$

# SAML requests.
^/_matrix/client/(api/v1|r0|unstable)/login/sso/redirect$
^/_matrix/saml2/authn_response$

# CAS requests.
^/_matrix/client/(api/v1|r0|unstable)/login/(cas|sso)/redirect$
^/_matrix/client/(api/v1|r0|unstable)/login/cas/ticket$

However, none of these endpoints are actually enabled on worker processes, so any attempts to send them to a worker will result in a 404.

@erikjohnston erikjohnston added the A-Docs things relating to the documentation label Jan 8, 2021
@richvdh richvdh changed the title workers.md gives incorrect advice about SSO endpoints workers.md gives incorrect advice about various endpoints Jan 28, 2021
@richvdh
Copy link
Member Author

richvdh commented Jan 28, 2021

Also: /_synapse/client/password_reset/email/submit_token

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Docs things relating to the documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants