darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2025-12-24 01:11:38 -05:00
Code Issues Packages Projects Releases Wiki Activity

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

Browse Source 
debug messages
This commit is contained in:
Carlo Costino
2024-01-09 17:52:16 -05:00
committed by GitHub
parent 54bd9b7e27 572c8ebb84
commit 653dfaf53b
2 changed files with 19 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/delivery/send_to_providers.py
View File

@@ -82,6 +82,10 @@ 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

17
app/user/rest.py
View File

@@ -353,12 +353,25 @@ def create_2fa_code(
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.set(key, recipient)
stored_recipient = redis_store.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!")
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!")
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: