From 29567729c74478e7c99e7d030cec6f83b635aa70 Mon Sep 17 00:00:00 2001 From: Adam Moussa Date: Mon, 27 Jul 2026 13:38:27 -0400 Subject: [PATCH] fix(logging): drop employee-derived DNs from forwarding log 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. --- src/shared/shared/three_cx_client.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/shared/shared/three_cx_client.py b/src/shared/shared/three_cx_client.py index b07103c..8dde050 100644 --- a/src/shared/shared/three_cx_client.py +++ b/src/shared/shared/three_cx_client.py @@ -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: