diff options
author | Andrew Morgan <1342360+anoadragon453@users.noreply.github.com> | 2021-01-27 11:49:31 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-01-27 11:49:31 +0000 |
commit | a64c29926efd8460dfc9561d761898197638973d (patch) | |
tree | 31ba71f57be3339a4e115cbf429dba309b37ccbd /changelog.d | |
parent | Add type hints to various handlers. (#9223) (diff) | |
download | synapse-a64c29926efd8460dfc9561d761898197638973d.tar.xz |
Pass a dict, instead of None, to modules if a None config is specified in the homeserver config (#9229)
If a Synapse module's config block were empty in YAML, thus being translated to a `Nonetype` in Python, then some modules could fail as that None ends up getting passed to their `parse_config` method. Modules are expected to accept a `dict` instead. This PR ensures that if the user does end up specifying an empty config block (such as what [the default oidc config in the sample config](https://github.com/matrix-org/synapse/blob/5310808d3bebd17275355ecd474bc013e8c7462d/docs/sample_config.yaml#L1816-L1845) states) then `None` is not passed to the module. An empty dict is passed instead. This code assumes that no existing modules are relying on receiving a `None` config block, but I'd really hope that they aren't.
Diffstat (limited to 'changelog.d')
-rw-r--r-- | changelog.d/9229.bugfix | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/changelog.d/9229.bugfix b/changelog.d/9229.bugfix new file mode 100644 index 0000000000..3ed32291de --- /dev/null +++ b/changelog.d/9229.bugfix @@ -0,0 +1 @@ +Fix a bug where `None` was passed to Synapse modules instead of an empty dictionary if an empty module `config` block was provided in the homeserver config. \ No newline at end of file |