search

pallets-eco / flask-security

Quick and simple security for Flask applications
MIT License
1.63k stars 513 forks source link

Fix problem with LocalProxy for email_sender #786

Closed fmerges closed 6 years ago

fmerges commented 6 years ago

This pull request is to solve the bug #782

avilaton commented 6 years ago

What problem is this solving? Would you please add a failing test that this fixes?

fmerges commented 6 years ago

Sorry my bad, I forgot to add the issue this pull is related to.

jirikuncar commented 6 years ago

@fmerges can you please rebase your PR on top of develop branch?

fmerges commented 6 years ago

Just noticed that on the develop branch this problem is already fixed. So the PR for development branch is not needed. See:

https://github.com/mattupstate/flask-security/blob/89198288bc416a7921b4973d8e897993942bb428/flask_security/utils.py#L385-L387

It's just in the current master where it is broken...