github/flagger
1
0
Fork 0
mirror of https://github.com/fluxcd/flagger.git synced 2026-09-05 02:47:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1887 from mtfurlan/fix/msteams-webhook-status-code

Browse Source 
fix(ms teams notifier): handle 202 status code
This commit is contained in:
Sanskar Jaiswal
2026-04-17 11:24:21 +05:30
committed by GitHub
parent 5d91ff55bd e7375e5ee0
commit f7e42a8b48
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pkg/notifier/client.go
+3 -3
View File
@@ -73,14 +73,14 @@ func postMessage(address, token, proxy string, payload interface{}) error {
res, err := httpClient.Do(req.WithContext(ctx))
if err != nil {
return fmt.Errorf("sending notification failed: %w", err)
return fmt.Errorf("sending notification failed to send: %w", err)
}
defer res.Body.Close()
statusCode := res.StatusCode
if statusCode != 200 {
if statusCode < 200 || 300 <= statusCode {
body, _ := io.ReadAll(res.Body)
return fmt.Errorf("sending notification failed: %s", string(body))
return fmt.Errorf("sending notification bad response %d: %s", statusCode, string(body))
}
return nil