You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
fix(slack-notifier): exclude health checks from alerts (#1335)
<!--- Provide a general summary of your changes in the Title above -->
## Description
<!--- Describe your changes in detail -->
Logging has changed a bit so need to change the endpoint to exclude from
alerts. We don't want to log the health-check endpoint traces, but that
has to be tweaked in another PR
## Related Issue(s)
- N/A
Slack discussion:
https://digdir.slack.com/archives/C079XRW5G5A/p1729643530959569
## Verification
- [ ] **Your** code builds clean without any errors or warnings
- [ ] Manual testing done (required)
- [ ] Relevant automated test added (if you find this hard, leave it and
we'll help out)
## Documentation
- [ ] Documentation is updated (either in `docs`-directory, Altinnpedia
or a separate linked PR in
[altinn-studio-docs.](https://github.com/Altinn/altinn-studio-docs), if
applicable)
<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit
- **New Features**
- Enhanced alerting logic for the Slack Notifier function app to broaden
the scope of monitored requests.
- Defined SKU parameters to ensure valid options for resource
deployment.
- **Bug Fixes**
- Adjusted criteria for exception alerting to improve monitoring
accuracy.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
0 commit comments