diff options
author | Christopher Allan Webber <cwebber@dustycloud.org> | 2011-09-07 23:32:15 -0500 |
---|---|---|
committer | Christopher Allan Webber <cwebber@dustycloud.org> | 2011-09-07 23:32:15 -0500 |
commit | 73fffbb8b0b37d642f7dc996bbec8fdf7d4e3e8b (patch) | |
tree | c57aca2c08f1adf6ff6c0e22efb7e8840b13fcfd | |
parent | 2c9e8184a44336689b8d3e0d01b17b15dfbcb791 (diff) | |
download | mediagoblin-73fffbb8b0b37d642f7dc996bbec8fdf7d4e3e8b.tar.lz mediagoblin-73fffbb8b0b37d642f7dc996bbec8fdf7d4e3e8b.tar.xz mediagoblin-73fffbb8b0b37d642f7dc996bbec8fdf7d4e3e8b.zip |
Adding additional check that verification key exists, and updating indentation
-rw-r--r-- | mediagoblin/auth/views.py | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/mediagoblin/auth/views.py b/mediagoblin/auth/views.py index 098443b8..dd693892 100644 --- a/mediagoblin/auth/views.py +++ b/mediagoblin/auth/views.py @@ -242,8 +242,10 @@ def verify_forgot_password(request): return render_404(request) # check if we have a real user and correct token - if (user and user['fp_verification_key'] == unicode(session_token) and - datetime.datetime.now() < user['fp_token_expire']): + if ((user and user['fp_verification_key'] and + user['fp_verification_key'] == unicode(session_token) and + datetime.datetime.now() < user['fp_token_expire'])): + cp_form = auth_forms.ChangePassForm(session_vars) if request.method == 'POST' and cp_form.validate(): @@ -255,9 +257,11 @@ def verify_forgot_password(request): return redirect(request, 'mediagoblin.auth.fp_changed_success') else: - return render_to_response(request, - 'mediagoblin/auth/change_fp.html', - {'cp_form': cp_form}) + return render_to_response( + request, + 'mediagoblin/auth/change_fp.html', + {'cp_form': cp_form}) + # in case there is a valid id but no user whit that id in the db # or the token expired else: |