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 #2901 from alphagov/turn-off-sync-statsd-for-posts

Browse Source 
Turn off statsd wrapper for synchronous statsd calls during POSTs
This commit is contained in:
David McDonald
2020-06-29 12:47:49 +01:00
committed by GitHub
parent a0a5cde15e 12f460adc5
commit 7d68280445
5 changed files with 6 additions and 36 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
app/__init__.py
View File

@@ -269,10 +269,6 @@ def register_v2_blueprints(application):
def init_app(app):
@app.before_request
def record_user_agent():
statsd_client.incr("user-agent.{}".format(process_user_agent(request.headers.get('User-Agent', None))))
@app.before_request
def record_request_details():
CONCURRENT_REQUESTS.inc()
@@ -319,18 +315,6 @@ def create_random_identifier():
return ''.join(random.choice(string.ascii_uppercase + string.digits) for _ in range(16))
def process_user_agent(user_agent_string):
if user_agent_string and user_agent_string.lower().startswith("notify"):
components = user_agent_string.split("/")
client_name = components[0].lower()
client_version = components[1].replace(".", "-")
return "{}.{}".format(client_name, client_version)
elif user_agent_string and not user_agent_string.lower().startswith("notify"):
return "non-notify-user-agent"
else:
return "unknown"
def setup_sqlalchemy_events(app):
TOTAL_DB_CONNECTIONS = Gauge(

1
app/dao/notifications_dao.py
View File

@@ -77,7 +77,6 @@ def dao_get_last_date_template_was_used(template_id, service_id):
return last_date
@statsd(namespace="dao")
@transactional
def dao_create_notification(notification):
if not notification.id:

2
requirements-app.txt
View File

@@ -27,7 +27,7 @@ notifications-python-client==5.5.1
# PaaS
awscli-cwlogs>=1.4,<1.5
git+https://github.com/alphagov/notifications-utils.git@39.7.1#egg=notifications-utils==39.7.1
git+https://github.com/alphagov/notifications-utils.git@40.0.0#egg=notifications-utils==40.0.0
# gds-metrics requires prometheseus 0.2.0, override that requirement as 0.7.1 brings significant performance gains
prometheus-client==0.7.1

10
requirements.txt
View File

@@ -29,7 +29,7 @@ notifications-python-client==5.5.1
# PaaS
awscli-cwlogs>=1.4,<1.5
git+https://github.com/alphagov/notifications-utils.git@39.7.1#egg=notifications-utils==39.7.1
git+https://github.com/alphagov/notifications-utils.git@40.0.0#egg=notifications-utils==40.0.0
# gds-metrics requires prometheseus 0.2.0, override that requirement as 0.7.1 brings significant performance gains
prometheus-client==0.7.1
@@ -40,14 +40,14 @@ alembic==1.4.2
amqp==1.4.9
anyjson==0.3.3
attrs==19.3.0
awscli==1.18.85
awscli==1.18.89
bcrypt==3.1.7
billiard==3.3.0.23
bleach==3.1.4
blinker==1.4
boto==2.49.0
boto3==1.10.38
botocore==1.17.8
botocore==1.17.12
certifi==2020.6.20
chardet==3.0.4
click==7.1.2
@@ -58,8 +58,8 @@ flask-redis==0.4.0
future==0.18.2
govuk-bank-holidays==0.6
greenlet==0.4.16
idna==2.9
importlib-metadata==1.6.1
idna==2.10
importlib-metadata==1.7.0
Jinja2==2.11.2
jmespath==0.10.0
kombu==3.0.37

13
tests/app/test_user_agent_processing.py
View File

@@ -1,13 +0,0 @@
from app import process_user_agent
def test_can_process_notify_api_user_agent():
assert "notify-api-python-client.3-0-0" == process_user_agent("NOTIFY-API-PYTHON-CLIENT/3.0.0")
def test_can_handle_non_notify_api_user_agent():
assert "non-notify-user-agent" == process_user_agent("Mozilla/5.0 (iPad; U; CPU OS 3_2_1 like Mac OS X; en-us) AppleWebKit/531.21.10 (KHTML, like Gecko) Mobile/7B405") # noqa
def test_handles_null_user_agent():
assert "unknown" == process_user_agent(None)