darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-02-05 10:53:28 -05:00
Code Issues Packages Projects Releases Wiki Activity

Fix bug with send_verify_code not including the to field.

Browse Source
This commit is contained in:
Nicholas Staples
2016-02-22 12:33:59 +00:00
parent b4ddcaa9b1
commit 980c01e10c
7 changed files with 11 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/main/dao/users_dao.py
View File

@@ -57,8 +57,8 @@ def request_password_reset(user):
user_api_client.update_user(user)
def send_verify_code(user_id, code_type, to=None):
return user_api_client.send_verify_code(user_id, code_type, to=to)
def send_verify_code(user_id, code_type, to):
return user_api_client.send_verify_code(user_id, code_type, to)
def check_verify_code(user_id, code, code_type):

2
app/main/views/code_not_received.py
View File

@@ -43,5 +43,5 @@ def verification_code_not_received():
def check_and_resend_verification_code():
# TODO there needs to be a way to generate a new session id
user = users_dao.get_user_by_email(session['user_details']['email'])
users_dao.send_verify_code(user.id, 'sms')
users_dao.send_verify_code(user.id, 'sms', user.mobile_number)
return redirect(url_for('main.two_factor'))

2
app/main/views/new_password.py
View File

@@ -21,7 +21,7 @@ def new_password(token):
form = NewPasswordForm()
if form.validate_on_submit():
users_dao.send_verify_code(user.id, 'sms')
users_dao.send_verify_code(user.id, 'sms', user.mobile_number)
session['user_details'] = {
'id': user.id,
'email': user.email_address,

4
app/main/views/register.py
View File

@@ -45,8 +45,8 @@ def register():
# How do we report to the user there is a problem with
# sending codes apart from service unavailable?
# at the moment i believe http 500 is fine.
users_dao.send_verify_code(user.id, 'sms')
users_dao.send_verify_code(user.id, 'email')
users_dao.send_verify_code(user.id, 'sms', user.mobile_number)
users_dao.send_verify_code(user.id, 'email', user.email_address)
session['expiry_date'] = str(datetime.now() + timedelta(hours=1))
session['user_details'] = {"email": user.email_address, "id": user.id}
return redirect(url_for('main.verify'))

2
app/main/views/sign_in.py
View File

@@ -27,7 +27,7 @@ def sign_in():
if user.state == 'pending':
return redirect(url_for('.verify'))
elif user.is_active():
users_dao.send_verify_code(user.id, 'sms')
users_dao.send_verify_code(user.id, 'sms', user.mobile_number)
return redirect(url_for('.two_factor'))
# Vague error message for login in case of user not known, locked, inactive or password not verified
flash('Username or password is incorrect')

4
app/main/views/user_profile.py
View File

@@ -74,7 +74,7 @@ def user_profile_email_authenticate():
if form.validate_on_submit():
session[NEW_EMAIL_PASSWORD_CONFIRMED] = True
send_verify_code(current_user.id, 'email', to=session[NEW_EMAIL])
send_verify_code(current_user.id, 'email', session[NEW_EMAIL])
return redirect(url_for('.user_profile_email_confirm'))
return render_template(
@@ -142,7 +142,7 @@ def user_profile_mobile_number_authenticate():
if form.validate_on_submit():
session[NEW_MOBILE_PASSWORD_CONFIRMED] = True
send_verify_code(current_user.id, 'sms', to=session[NEW_MOBILE])
send_verify_code(current_user.id, 'sms', session[NEW_MOBILE])
return redirect(url_for('.user_profile_mobile_number_confirm'))
return render_template(