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

Merge pull request #1142 from alphagov/log-ip-block

Browse Source 
log ip address as separate field to allow analysis of messages
This commit is contained in:
Leo Hemsted
2017-08-22 17:01:54 +01:00
committed by GitHub
parent 843812cbe9 74cd1b2904
commit 12cf36f6fa
1 changed files with 10 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
app/authentication/auth.py
View File

@@ -59,13 +59,16 @@ def restrict_ip_sms():
if request.headers.get("X-Custom-forwarder"): if request.headers.get("X-Custom-forwarder"):
current_app.logger.info("X-Custom-forwarder {}".format(request.headers.get("X-Custom-forwarder"))) current_app.logger.info("X-Custom-forwarder {}".format(request.headers.get("X-Custom-forwarder")))
if ip in current_app.config.get('SMS_INBOUND_WHITELIST'): current_app.logger.info({
current_app.logger.info("Inbound sms ip addresses {} passed ".format(ip)) 'message': 'Inbound sms ip address',
return 'log_contents': {
else: 'passed': ip in current_app.config.get('SMS_INBOUND_WHITELIST'),
current_app.logger.info("Inbound sms ip addresses blocked {}".format(ip)) 'ip_address': ip
return }
# raise AuthError('Unknown source IP address from the SMS provider', 403) })
return
# raise AuthError('Unknown source IP address from the SMS provider', 403)
def requires_admin_auth(): def requires_admin_auth():