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

Moving to Flask-Security-Fork, other updates #482

Merged
merged 1 commit into from Dec 30, 2016

Conversation

Projects
None yet
2 participants
@scriptsrc
Copy link
Contributor

commented Dec 30, 2016

  • Replacing Flask-Security with Flask-Security-Fork.
  • Removing some explicit dependencies to rely on flask-security-fork dependencies.
  • SSO will use flask-security login_user instead of flask-login so that security_trackable works.
  • Replacing current_user.is_authenticated() method with property so we can use a newer version of flask-login.
  • Bumping psycopg2, bcrypt, and cryptography versions.
Replacing Flask-Security with Flask-Security-Fork. Removing some expl…
…icit dependencies to rely on flask-security-fork dependencies. SSO will use flask-security login_user instead of flask-login so that security_trackable works. Replacing current_user.is_authenticated() method with property so we can use a newer version of flask-login.
@scriptsrc

This comment has been minimized.

Copy link
Contributor Author

commented Dec 30, 2016

Big reason to move to Flask-Security-Fork is mattupstate/flask-security#548

Without PR 548 being merged, it's vulnerable to logout redirection. Flask-Security-Fork is still receiving support.

@coveralls

This comment has been minimized.

Copy link

commented Dec 30, 2016

Coverage Status

Coverage remained the same at 51.515% when pulling 51ca485 on update_some_dependencies into da8c81e on develop.

@scriptsrc

This comment has been minimized.

Copy link
Contributor Author

commented Dec 30, 2016

pip uninstall flask-security && pip install flask-security-fork

@scriptsrc scriptsrc merged commit 3b4da13 into develop Dec 30, 2016

3 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 51.515%
Details

@scriptsrc scriptsrc removed the in progress label Dec 30, 2016

@scriptsrc scriptsrc referenced this pull request Jan 13, 2017

Merged

Release v0.8.0 #458

@scriptsrc scriptsrc deleted the update_some_dependencies branch Feb 17, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.