Skip to content

feat(health): add independent notification provider health checks (#709) - #739

Open
ravendevhub wants to merge 1 commit into
Core-Foundry:mainfrom
ravendevhub:feat/provider-health-checks-709
Open

feat(health): add independent notification provider health checks (#709)#739
ravendevhub wants to merge 1 commit into
Core-Foundry:mainfrom
ravendevhub:feat/provider-health-checks-709

Conversation

@ravendevhub

Copy link
Copy Markdown

Summary

Resolves #709 by introducing an independent health inspection module (listener/src/services/provider-health-monitor.ts) enabling operators to monitor the availability, response latency, and status of external notification destinations (Webhooks, Discord) without ever exposing tokens or credentials in diagnostic outputs.

Changes

  1. Provider Health Monitor (listener/src/services/provider-health-monitor.ts):
    • Independently probes destination endpoints with configurable timeouts.
    • Strips and masks Discord webhook tokens and basic auth credentials from URLs.
    • Provides aggregated health reporting with status classifications (healthy | degraded | unhealthy | disabled).
  2. Automated Unit Tests (listener/src/services/provider-health-monitor.test.ts):
    • Tests URL sanitization, response code evaluation, timeout handling, and multi-provider aggregation.
  3. Documentation (docs/PROVIDER_HEALTH_CHECKS.md):
    • Documents diagnostic data format and operational architecture.

Acceptance Criteria

  • Provider health can be checked independently.
  • External failures are reported without exposing credentials.
  • Health information is available through an appropriate diagnostic interface.

…re-Foundry#709)

- Implement provider health monitor checking Webhooks, Discord, and endpoints independently
- Sanitize and redact credentials and webhook tokens from diagnostic targets and errors
- Provide aggregated health reports with latency metrics and status breakdown
- Add unit test suite in provider-health-monitor.test.ts and docs in docs/PROVIDER_HEALTH_CHECKS.md
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.

Add Notification Provider Health Checks

1 participant