search

pallets-eco / flask-security

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

reset_password: handle JSON requests (#453) #836

Closed mephi42 closed 2 years ago

mephi42 commented 5 years ago

Another stab at #468 and #535.

mephi42 commented 5 years ago

@lnielsen Could you please take a look at this PR?

mephi42 commented 5 years ago

@jwag956 this is my use case as well. Apparently quite a few people needed that in the past too, so I've decided to revive this topic.

@jirikuncar, @lnielsen, Gentle ping.

jwag956 commented 5 years ago

You can see my changes at: https://github.com/jwag956/flask-security/commits/develop

Also - in an effort to kickstart flask-security and get a release cadence going - I have formalized my fork - it is up on pypi, travis, readthedocs as flask-security-too.

My plan is to pick up obvious/easy already merged PRs from 'develop' branch to get started. Then improving my changes for json/APIs and getting those in.

jwag956 commented 5 years ago

I have merged my fix to this and confirmation, passwordless login that handle the case you are working on as well as other. In particular, for single-page-applications - the UI needs control on the GET on the reset password link. This will be part of my 3.2.0 release.

please see: https://github.com/jwag956/flask-security/pull/81

I will have this up on pypi as a rc candidate in a day or so.

mephi42 commented 5 years ago

That sounds really good, thanks! I will give it a try once it's on pypi.

jwag956 commented 5 years ago

@mephi42 It is now available as 3.2.0rc1 at: https://pypi.org/project/Flask-Security-Too/#history