From 462c8db4c34c9f70b7a06c2c9913eccef087c637 Mon Sep 17 00:00:00 2001 From: Alexey Bezhan Date: Tue, 27 Nov 2018 15:50:51 +0000 Subject: [PATCH] Lower service rate limit log level to INFO Services reaching rate limits are triggering our alerts and make it hard to find actual exceptions in the logs. As far as the API is concerned this is not an exceptional state, so we shouldn't log it as such. --- app/notifications/validators.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/notifications/validators.py b/app/notifications/validators.py index fb39c18fa..8fd129896 100644 --- a/app/notifications/validators.py +++ b/app/notifications/validators.py @@ -29,7 +29,7 @@ def check_service_over_api_rate_limit(service, api_key): rate_limit = service.rate_limit interval = 60 if redis_store.exceeded_rate_limit(cache_key, rate_limit, interval): - current_app.logger.error("service {} has been rate limited for throughput".format(service.id)) + current_app.logger.info("service {} has been rate limited for throughput".format(service.id)) raise RateLimitError(rate_limit, interval, api_key.key_type) @@ -41,7 +41,7 @@ def check_service_over_daily_message_limit(key_type, service): service_stats = services_dao.fetch_todays_total_message_count(service.id) redis_store.set(cache_key, service_stats, ex=3600) if int(service_stats) >= service.message_limit: - current_app.logger.error( + current_app.logger.info( "service {} has been rate limited for daily use sent {} limit {}".format( service.id, int(service_stats), service.message_limit) )