Matrix-Mirrors/synapse-product
1
0
Fork 0
mirror of https://git.anonymousland.org/anonymousland/synapse-product.git synced 2024-10-01 08:25:44 -04:00
Code Issues Packages Projects Releases Wiki Activity

Avoid attribute error when password_config present but empty (#6753)

Browse Source 
The old statement returned `None` for such a `password_config` (like the one
created on first run), thus retrieval of the `pepper` key failed with
`AttributeError`.

Fixes #5315

Signed-off-by: Ivan Vilata i Balaguer <ivan@selidor.net>
This commit is contained in:
Ivan Vilata-i-Balaguer 2020-01-22 02:32:52 -05:00 committed by Richard van der Hoff
parent 07124d028d
commit 837f62266b
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
changelog.d/6753.bugfix Normal file
View File

@ -0,0 +1 @@
Fix `AttributeError: 'NoneType' object has no attribute 'get'` in `hash_password` when configuration has an empty `password_config`. Contributed by @ivilata.

2
scripts/hash_password
View File

@ -52,7 +52,7 @@ if __name__ == "__main__":
if "config" in args and args.config: if "config" in args and args.config:
config = yaml.safe_load(args.config) config = yaml.safe_load(args.config)
bcrypt_rounds = config.get("bcrypt_rounds", bcrypt_rounds) bcrypt_rounds = config.get("bcrypt_rounds", bcrypt_rounds)
password_config = config.get("password_config", {}) password_config = config.get("password_config", None) or {}
password_pepper = password_config.get("pepper", password_pepper) password_pepper = password_config.get("pepper", password_pepper)
password = args.password password = args.password