adam/afterhours-shift-manager
1
0
Fork 0
mirror of https://github.com/Sea-Haven-Industries/afterhours-shift-manager.git synced 2026-09-30 06:43:12 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

fix(logging): drop employee-derived DNs from forwarding log

Browse source 
Resolves new code-scanning alerts 16/17. The closed/holiday DNs added
in the previous commit derive from roster employee lookups in the
Slack bot, so CodeQL classifies them as private data. Log only the
resource type; ring_scheduler already logs the queue number.
This commit is contained in:
Adam Moussa 2026-07-27 13:38:27 -04:00
parent 32bc75af57
commit 29567729c7
No known key found for this signature in database
1 changed files with 1 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/shared/shared/three_cx_client.py
View file

@ -102,9 +102,7 @@ class ThreeCXClient:
json=payload,
)
resp.raise_for_status()
logger.info(
"Updated %s forwarding closed=%s holiday=%s", resource, closed, holiday
)
logger.info("Updated %s forwarding", resource)
return resp.status_code
def get_ring_group(self, extension_number: str) -> dict: