pallets-eco / flask-security

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

Fix login and unified signin templates to send CSRF tokens. #825

Closed jwag956 closed 1 year ago

jwag956 commented 1 year ago

Add additional tests.

The oauthstart view needs to be decorated with @unauth_csrf.

Turn on complete CSRF support in view_scaffold since we really don't test CSRF enough.

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (a6d57f5) 98.15% compared to head (af984aa) 98.15%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #825 +/- ## ======================================= Coverage 98.15% 98.15% ======================================= Files 34 34 Lines 4453 4455 +2 ======================================= + Hits 4371 4373 +2 Misses 82 82 ``` | [Files Changed](https://app.codecov.io/gh/Flask-Middleware/flask-security/pull/825?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Flask-Middleware) | Coverage Δ | | |---|---|---| | [flask\_security/oauth\_glue.py](https://app.codecov.io/gh/Flask-Middleware/flask-security/pull/825?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Flask-Middleware#diff-Zmxhc2tfc2VjdXJpdHkvb2F1dGhfZ2x1ZS5weQ==) | `98.86% <100.00%> (+0.02%)` | :arrow_up: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.