matrix-org / synapse

Synapse: Matrix homeserver written in Python/Twisted.
https://matrix-org.github.io/synapse
Apache License 2.0
11.8k stars 2.13k forks source link

SAML: sp_config doesn't refresh IDP metadata #9598

Closed localguru closed 3 years ago

localguru commented 3 years ago

Description

If SAML is configured with remote metadata url, the IDP's metadata is only fetched at startup of synapse. If the IPD changes its certificate SAML login is broken on SP (synapse).

Steps to reproduce

If you are seeing this page after clicking a link sent to you via email, make sure you only click the confirmation link once, and that you open the validation link in the same client you're logging in from.

Try logging in again from your Matrix client and if the problem persists please contact the server's administrator.

Error: invalid_response


I am not sure whether to classify this as a bug or a feature request. But it should be possible for the SP to query the metadata of the IPD at regular intervals and renew it if necessary.

### Version information

Synapse 1.25, Ubuntu 18.04 LTS, debian packages from matrix.org
clokep commented 3 years ago

Would using mdq instead of remote work? (From the pysaml docs: https://pysaml2.readthedocs.io/en/latest/howto/config.html#metadata) I'm not sure what the difference is unfortunately.

richvdh commented 3 years ago

closing this pending the requested info.