pallets-eco / flask-security

Quick and simple security for Flask applications
MIT License
635 stars 155 forks source link

Change confirmation to default to NOT auto-login. #820

Closed jwag956 closed 1 year ago

jwag956 commented 1 year ago

This makes it consistent with /reset and with OWASP best practices.

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.03 :warning:

Comparison is base (2750b9a) 98.15% compared to head (379b68c) 98.13%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #820 +/- ## ========================================== - Coverage 98.15% 98.13% -0.03% ========================================== Files 34 34 Lines 4456 4458 +2 ========================================== + Hits 4374 4375 +1 - Misses 82 83 +1 ``` | [Impacted Files](https://app.codecov.io/gh/Flask-Middleware/flask-security/pull/820?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Flask-Middleware) | Coverage Δ | | |---|---|---| | [flask\_security/views.py](https://app.codecov.io/gh/Flask-Middleware/flask-security/pull/820?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Flask-Middleware#diff-Zmxhc2tfc2VjdXJpdHkvdmlld3MucHk=) | `98.56% <ø> (-0.21%)` | :arrow_down: | | [flask\_security/core.py](https://app.codecov.io/gh/Flask-Middleware/flask-security/pull/820?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Flask-Middleware#diff-Zmxhc2tfc2VjdXJpdHkvY29yZS5weQ==) | `98.61% <100.00%> (+<0.01%)` | :arrow_up: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.