paultanay opened a new pull request, #4350: URL: https://github.com/apache/hertzbeat/pull/4350
## What's changed Fixes #4349 This PR fixes the fallback default Gotify webhook URL in `AlerterProperties`. `GotifyAlertNotifyHandlerImpl` builds the webhook URL with `String.format(...)`, but the Java fallback default was missing the `%s` placeholder. When that fallback value is used, the receiver Gotify token is ignored and the generated request URL does not include the token. This change makes the Java fallback default consistent with the configured value in `application.yml`. ## Changes - Added the missing `%s` token placeholder to the default Gotify webhook URL. - Added/updated unit test coverage for Gotify URL formatting. ## Test ```bash mvn -pl hertzbeat-alerter -Dtest=GotifyAlertNotifyHandlerImplTest test ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
