Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for not using on AnonymousUser. #877

Merged
merged 1 commit into from
Nov 10, 2023
Merged

Add support for not using on AnonymousUser. #877

merged 1 commit into from
Nov 10, 2023

Conversation

jwag956
Copy link
Collaborator

@jwag956 jwag956 commented Nov 9, 2023

Flask-Login is proposing doing away with AnonymousUser concept - we now support that as well. There will likely be some small changes required when they decide precisely how they plan on implementing it.

Add a new utility method is_user_authenticated that encapsulates both models. Add a new template context method _fs_is_user_authenticated for use in templates.

Copy link

codecov bot commented Nov 9, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Files Coverage Δ
flask_security/__init__.py 100.00% <100.00%> (ø)
flask_security/cli.py 99.41% <100.00%> (+<0.01%) ⬆️
flask_security/core.py 98.61% <100.00%> (+0.19%) ⬆️
flask_security/decorators.py 96.23% <100.00%> (ø)
flask_security/forms.py 99.21% <100.00%> (+<0.01%) ⬆️
flask_security/unified_signin.py 99.00% <100.00%> (ø)
flask_security/utils.py 97.60% <100.00%> (+0.02%) ⬆️
flask_security/views.py 98.76% <100.00%> (ø)

📢 Thoughts on this report? Let us know!

Flask-Login is proposing doing away with AnonymousUser concept - we now support that as well.
There will likely be some small changes required when they decide precisely how they plan on implementing it.

Add a new utility method is_user_authenticated that encapsulates both models.
Add a new template context method _fs_is_user_authenticated for use in templates.

Add support (not currently used) to set configuration settings from pytest command line.
@jwag956 jwag956 merged commit 6d353db into master Nov 10, 2023
16 checks passed
@jwag956 jwag956 deleted the lesslm branch November 10, 2023 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant