fix: webhook notificator: fix null ptr

wuriyanto48 · wuriyanto48 · commit 8fd218c03460 · 2025-04-23T15:22:31.000+07:00
diff --git a/notificators/webhook/webhook.go b/notificators/webhook/webhook.go
@@ -116,12 +116,12 @@ func (d *Webhook) Send(msg string) error {
 					if d.verbose {
 						d.logger.Println(err)
 					}
-				}
-
-				statusOK := resp.StatusCode >= 200 && resp.StatusCode < 300
-				if !statusOK {
-					if d.verbose {
-						d.logger.Printf("notificator %s error: %s\n", d.Provider(), httpx.ErrorStatusNot200)
+				} else {
+					statusOK := resp.StatusCode >= 200 && resp.StatusCode < 300
+					if !statusOK {
+						if d.verbose {
+							d.logger.Printf("notificator %s error: %s\n", d.Provider(), httpx.ErrorStatusNot200)
+						}
 					}
 				}
 			}(conf.webhookURL, bytes.NewBuffer(messageJSON), conf.headers, 5, d)

Original file line number	Diff line number	Diff line change
`@@ -116,12 +116,12 @@ func (d *Webhook) Send(msg string) error {`
`116`	`116`	`if d.verbose {`
`117`	`117`	`d.logger.Println(err)`
`118`	`118`	`}`
`119`		`- }`
`120`		`-`
`121`		`- statusOK := resp.StatusCode >= 200 && resp.StatusCode < 300`
`122`		`- if !statusOK {`
`123`		`- if d.verbose {`
`124`		`- d.logger.Printf("notificator %s error: %s\n", d.Provider(), httpx.ErrorStatusNot200)`
	`119`	`+ } else {`
	`120`	`+ statusOK := resp.StatusCode >= 200 && resp.StatusCode < 300`
	`121`	`+ if !statusOK {`
	`122`	`+ if d.verbose {`
	`123`	`+ d.logger.Printf("notificator %s error: %s\n", d.Provider(), httpx.ErrorStatusNot200)`
	`124`	`+ }`
`125`	`125`	`}`
`126`	`126`	`}`
`127`	`127`	`}(conf.webhookURL, bytes.NewBuffer(messageJSON), conf.headers, 5, d)`