Merge pull request #28401 from barredterra/wrong-enc-key
fix(Login): don't prevent login if encryption key is invalid
This commit is contained in:
commit
504ef2e9e4
2 changed files with 10 additions and 2 deletions
|
|
@ -34,7 +34,13 @@ def get_decrypted_password(doctype, name, fieldname="password", raise_exception=
|
|||
).run()
|
||||
|
||||
if result and result[0][0]:
|
||||
return decrypt(result[0][0], key=f"{doctype}.{name}.{fieldname}")
|
||||
try:
|
||||
return decrypt(result[0][0], key=f"{doctype}.{name}.{fieldname}")
|
||||
except frappe.ValidationError as e:
|
||||
if raise_exception:
|
||||
raise e
|
||||
|
||||
return None
|
||||
|
||||
elif raise_exception:
|
||||
frappe.throw(
|
||||
|
|
|
|||
|
|
@ -69,7 +69,9 @@ def get_context(context):
|
|||
)
|
||||
|
||||
for provider in providers:
|
||||
client_secret = get_decrypted_password("Social Login Key", provider.name, "client_secret")
|
||||
client_secret = get_decrypted_password(
|
||||
"Social Login Key", provider.name, "client_secret", raise_exception=False
|
||||
)
|
||||
if not client_secret:
|
||||
continue
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue