darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2025-12-30 20:31:37 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #730 from GSA/notify-api-714

Browse Source 
remove debugging for 2fa issue
This commit is contained in:
Carlo Costino
2024-01-11 13:00:36 -05:00
committed by GitHub
parent 4533edbb5d a8955ef78f
commit 8cc582afb9
2 changed files with 1 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
app/delivery/send_to_providers.py
View File

@@ -82,10 +82,7 @@ def send_sms_to_provider(notification):
if my_phone:
my_phone = my_phone.decode("utf-8")
# TODO REMOVE
current_app.logger.info(
f"IN SEND TO PROVIDERS, WHERE WE GET THE VALUE, KEY IS {key} and value is {my_phone}"
)
if my_phone is None:
si = notification.service_id
ji = notification.job_id

23
app/user/rest.py
View File

@@ -351,33 +351,10 @@ def create_2fa_code(
reply_to_text=reply_to,
)
current_app.logger.info("TESTING REDIS")
redis_store.raw_set("TESTKEY5", "WORKS")
current_app.logger.info(
f"SHOULD SEE THE WORD 'WORKS' HERE: {redis_store.get('TESTKEY5')}"
)
key = f"2facode-{saved_notification.id}".replace(" ", "")
recipient = str(recipient)
# TODO REMOVE
current_app.logger.info(
f"IN REST, WHERE WE SET THE VALUE, KEY IS {key} and value is {recipient}"
)
redis_store.raw_set(key, recipient)
stored_recipient = redis_store.raw_get(key)
# TODO REMOVE
current_app.logger.info(
f"IN REST, WHERE WE GET THE VALUE, KEY IS {key} and value is {stored_recipient}"
)
if stored_recipient:
current_app.logger.info(
"IN user/rest.py we saved the recipient of the 2facode to redis!"
)
else:
current_app.logger.info(
"IN user/rest.py we did NOT save the recipient of the 2facode to redis!"
)
# Assume that we never want to observe the Notify service's research mode
# setting for this notification - we still need to be able to log into the
# admin even if we're doing user research using this service: