Skip to content

fix: redact IVR menu choice logs - #252

Merged
Justinabox merged 1 commit into
mainfrom
agent/redact-ivr-choice-logs-20260727
Jul 28, 2026
Merged

fix: redact IVR menu choice logs#252
Justinabox merged 1 commit into
mainfrom
agent/redact-ivr-choice-logs-20260727

Conversation

@Justinabox

Copy link
Copy Markdown
Owner

Summary

  • Redact valid IVR menu-selection INFO logs so they never include the selected DTMF value or application-supplied option description.
  • Redact invalid menu-choice DEBUG logs while retaining accepted/invalid operational signals.
  • Add focused caplog regressions that prove routing and handler behavior remain intact and synthetic sensitive values never enter logs.

Issue

Closes #61 (narrow IVR logging slice; does not overlap the separate DTMF collector work in #238).

Verification

  • git diff --check
  • PYTHONPATH=. uv run --no-project --with pytest --with pytest-asyncio --with pyserial-asyncio pytest tests/test_ivr.py -q → 16 passed
  • PYTHONPATH=. uv run --no-project --with pytest --with pytest-asyncio --with pytest-aiohttp --with pyserial-asyncio --with aiosqlite pytest tests/ -q → 993 passed

Risk

Low: this changes only log payloads. IVR routing, return values, handler invocation, retry behavior, and audio prompts are unchanged. No hardware or external backend access was used.

@Justinabox

Copy link
Copy Markdown
Owner Author

Release-gate evidence: reviewed the IVR logging-only privacy change and its runtime caplog coverage. I merge-tested this PR head with current origin/main after #254; git diff --check passed and the required local gate passed: PYTHONPATH=. uv run --no-project --with pytest --with pytest-asyncio --with pytest-aiohttp --with pyserial-asyncio --with aiosqlite pytest tests/ -q → 997 passed. No GitHub check runs are configured. Classified safe-to-land; proceeding with squash merge.

@Justinabox
Justinabox marked this pull request as ready for review July 28, 2026 02:22
@Justinabox
Justinabox merged commit 3a5ca81 into main Jul 28, 2026
@Justinabox
Justinabox deleted the agent/redact-ivr-choice-logs-20260727 branch July 28, 2026 02:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Redact phone numbers and webhook details from default logs

1 participant