Skip to content

Conversation

@gluckzhang
Copy link
Member

Summary

  • Added disable_resolve_message configuration option to webhook alerts
  • When enabled, only firing alerts are sent; resolved notifications are skipped
  • Useful for systems that only need notifications when issues occur
  • Includes comprehensive test coverage for all scenarios (enabled, disabled, default)

Test Plan

  • Run go test ./td2/ to verify all tests pass
  • All 7 test cases in TestNotifyWebhook pass (3 existing + 4 new)

🤖 Generated with Claude Code

This change adds a new configuration option to suppress resolved alert
messages sent to webhook endpoints. When disable_resolve_message is set
to yes, only firing alerts are sent, which is useful for systems that
only need notifications when issues occur. Includes comprehensive test
coverage for all configuration scenarios.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@gluckzhang gluckzhang merged commit aaf93b0 into main Jan 20, 2026
2 checks passed
@gluckzhang gluckzhang deleted the gluckzhang/disable-resolve-msg branch January 20, 2026 12:52
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