Skip to content

test(alerts): add deeply nested collection null byte sanitization specs - #6802

Closed
gcoinstash-cmd wants to merge 1 commit into
keephq:mainfrom
gcoinstash-cmd:test/day2-w2-sanitize-alert-deep-nesting-specs
Closed

gcoinstash-cmd wants to merge 1 commit into
keephq:mainfrom
gcoinstash-cmd:test/day2-w2-sanitize-alert-deep-nesting-specs

Conversation

@gcoinstash-cmd

Copy link
Copy Markdown

Summary

  • Extends test parameters in tests/test_alert_utils.py to verify that sanitize_alert recursively strips null bytes (\x00) across deeply nested dictionaries containing embedded lists of objects.
  • Adds regression protection for empty dictionary inputs.

Testing

  • Verifies recursive sanitization and structural equality.

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

No linked issues found. Please add the corresponding issues in the pull request description.
Use GitHub automation to close the issue when a PR is merged

@shahargl

shahargl commented Sep 9, 2026

Copy link
Copy Markdown
Member

ai slop

@shahargl shahargl closed this Sep 9, 2026
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.

2 participants