darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-08-15 15:28:02 -04:00
Code Issues Packages Projects Releases Wiki Activity

fix phone number check

Browse Source
This commit is contained in:
Kenneth Kehl
2025-04-15 11:13:26 -07:00
parent f15d6128dc
commit 1f4b408d1d
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
app/delivery/send_to_providers.py
View File

@@ -105,11 +105,13 @@ def send_sms_to_provider(notification):
# The future home of the validation is TBD
_experimentally_validate_phone_numbers(recipient)
# We need to handle the case where people ignored the "1" country code for US
# East Timor (670) has ten digit numbers including code so requires special handling
# TODO current we allow US phone numbers to be uploaded without the country code (1)
# This will break certain international phone numbers (Norway, Denmark, East Timor)
# When we officially announce support for international numbers, US numbers must contain
# their country code.
recipient = str(recipient)
if len(recipient) == 10 and not recipient.startswith("670"):
if os.getenv("NOTIFY_ENVIRONMENT") not in ["test"]:
if len(recipient) == 10:
if os.getenv("NOTIFY_ENVIRONMENT") not in ["test"]: # we want to test intl support
recipient = f"1{recipient}"
sender_numbers = get_sender_numbers(notification)