Skip to content
This repository has been archived by the owner on Aug 23, 2020. It is now read-only.

Test against Django 1.11 #81

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

joshblum
Copy link

@joshblum joshblum commented Jun 8, 2017

Part of the fix for #80 can be done by using class-based views in Django 1.11

To avoid the possibility of leaking a password reset token via the HTTP Referer header (for example, if the reset page includes a reference to CSS or JavaScript hosted on another domain), the PasswordResetConfirmView (but not the deprecatedpassword_reset_confirm()function-based view) stores the token in a session and redirects to itself to present the password change form to the user without the token in the URL.

https://docs.djangoproject.com/en/1.11/releases/1.11/#django-contrib-auth

@coveralls
Copy link

coveralls commented Jun 8, 2017

Coverage Status

Coverage decreased (-2.2%) to 88.174% when pulling ef43c5c on joshblum:support-django-1.11 into 9a4adc6 on lambdalisue:master.

@joshblum joshblum force-pushed the support-django-1.11 branch from ef43c5c to e5ba0f3 Compare June 8, 2017 19:03
@coveralls
Copy link

Coverage Status

Coverage decreased (-2.2%) to 88.174% when pulling e5ba0f3 on joshblum:support-django-1.11 into 9a4adc6 on lambdalisue:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-2.2%) to 88.174% when pulling e5ba0f3 on joshblum:support-django-1.11 into 9a4adc6 on lambdalisue:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-2.2%) to 88.174% when pulling c44b74b on joshblum:support-django-1.11 into 9a4adc6 on lambdalisue:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-2.2%) to 88.174% when pulling c44b74b on joshblum:support-django-1.11 into 9a4adc6 on lambdalisue:master.

@coveralls
Copy link

coveralls commented Jun 8, 2017

Coverage Status

Coverage increased (+0.01%) to 90.424% when pulling de349f0 on joshblum:support-django-1.11 into 9a4adc6 on lambdalisue:master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants