diff options
| author | Richard van der Hoff <1389908+richvdh@users.noreply.github.com> | 2020-12-08 14:04:35 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-12-08 14:04:35 +0000 |
| commit | ab7a24cc6bbffa5ba67b42731c45b1d4d33f3ae3 (patch) | |
| tree | 9aeefb42b2c5871847505fc50f58127c7b605ebb /synapse/config/saml2_config.py | |
| parent | Simplify the flow for SSO UIA (#8881) (diff) | |
| download | synapse-ab7a24cc6bbffa5ba67b42731c45b1d4d33f3ae3.tar.xz | |
Better formatting for config errors from modules (#8874)
The idea is that the parse_config method of extension modules can raise either a ConfigError or a JsonValidationError, and it will be magically turned into a legible error message. There's a few components to it: * Separating the "path" and the "message" parts of a ConfigError, so that we can fiddle with the path bit to turn it into an absolute path. * Generally improving the way ConfigErrors get printed. * Passing in the config path to load_module so that it can wrap any exceptions that get caught appropriately.
Diffstat (limited to 'synapse/config/saml2_config.py')
| -rw-r--r-- | synapse/config/saml2_config.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/synapse/config/saml2_config.py b/synapse/config/saml2_config.py index c1b8e98ae0..7b97d4f114 100644 --- a/synapse/config/saml2_config.py +++ b/synapse/config/saml2_config.py @@ -125,7 +125,7 @@ class SAML2Config(Config): ( self.saml2_user_mapping_provider_class, self.saml2_user_mapping_provider_config, - ) = load_module(ump_dict) + ) = load_module(ump_dict, ("saml2_config", "user_mapping_provider")) # Ensure loaded user mapping module has defined all necessary methods # Note parse_config() is already checked during the call to load_module |