darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2025-12-20 07:21:13 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #979 from GSA/notify-api-956

Browse Source 
Remove log statement from s3.py
This commit is contained in:
Carlo Costino
2024-05-13 11:21:23 -04:00
committed by GitHub
parent 36f6621f71 e7183da117
commit e33421b62d
2 changed files with 3 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
app/aws/s3.py
View File

@@ -82,9 +82,6 @@ def incr_jobs_cache_misses():
redis_store.set(JOBS_CACHE_MISSES, 1)
else:
redis_store.incr(JOBS_CACHE_MISSES)
hits = redis_store.get(JOBS_CACHE_HITS).decode("utf-8")
misses = redis_store.get(JOBS_CACHE_MISSES).decode("utf-8")
current_app.logger.debug(f"JOBS CACHE MISS hits {hits} misses {misses}")
def incr_jobs_cache_hits():
@@ -92,9 +89,6 @@ def incr_jobs_cache_hits():
redis_store.set(JOBS_CACHE_HITS, 1)
else:
redis_store.incr(JOBS_CACHE_HITS)
hits = redis_store.get(JOBS_CACHE_HITS).decode("utf-8")
misses = redis_store.get(JOBS_CACHE_MISSES).decode("utf-8")
current_app.logger.debug(f"JOBS CACHE HIT hits {hits} misses {misses}")
def extract_phones(job):
@@ -102,7 +96,6 @@ def extract_phones(job):
first_row = job[0]
job.pop(0)
first_row = first_row.split(",")
current_app.logger.info(f"HEADERS {first_row}")
phone_index = 0
for item in first_row:
# Note: may contain a BOM and look like \ufeffphone number