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

Merge pull request #1284 from alphagov/vb-inbound-sms-ip

Browse Source 
Inbound sms ip messages
This commit is contained in:
Venus Bailey
2017-09-28 10:20:31 +01:00
committed by GitHub
parent 9a1aedec43 eb840585a7
commit e2bf6fde67
1 changed files with 5 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
app/authentication/auth.py
View File

@@ -70,11 +70,6 @@ def restrict_ip_sms():
for allowed_ip in allowed_ips for allowed_ip in allowed_ips
) )
if allowed:
return
else:
raise AuthError('Unknown source IP address from the SMS provider', 403)
current_app.logger.info({ current_app.logger.info({
'message': 'Inbound sms ip address', 'message': 'Inbound sms ip address',
'log_contents': { 'log_contents': {
@@ -82,11 +77,12 @@ def restrict_ip_sms():
'ip_address': inbound_ip 'ip_address': inbound_ip
} }
}) })
return
current_app.logger.error('Traffic from unknown source or route, X-Forwarded-For="{}"'.format( if allowed:
request.headers.get("X-Forwarded-For")) return
) else:
raise AuthError('Unknown source IP address from the SMS provider', 403)
raise AuthError('Traffic from unknown source or route', 403) raise AuthError('Traffic from unknown source or route', 403)