Skip to content

Standardization of Alerting Module - #303

Merged
Lorygold merged 1 commit into
certego:developfrom
kunalsz:alerting_standardization
Jun 16, 2025
Merged

Lorygold merged 1 commit into
certego:developfrom
kunalsz:alerting_standardization

Conversation

@kunalsz

@kunalsz kunalsz commented Jun 11, 2025

Copy link
Copy Markdown
Contributor

Resolves #302

Changes made:

  • Ensured that all the alerters have a similar structure
  • Ensured that all the tests of alerters have a similar structure
  • Moved all the necessary configs to the central config: alerting.json
  • Implemented error handling using try-except wherever necessary along with logging
  • Updated names of the alerters to follow a similar pattern

Signed-off-by: kunalsz <kunalavengers@gmail.com>
@ManofWax
ManofWax requested review from Lorygold and ManofWax June 12, 2025 07:28
@Lorygold
Lorygold merged commit 99027ce into certego:develop Jun 16, 2025
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.

[P1] Standardization of Alerting Module

3 participants