Merge pull request #393 from Juneezee/refactor/redundant-nil-check

lakhansamani · web-flow · commit 0428488dab16 · 2023-09-11T11:42:58.000+05:30
refactor(server/utils): remove redundant nil check
diff --git a/server/utils/webhook.go b/server/utils/webhook.go
@@ -83,10 +83,8 @@ func RegisterEvent(ctx context.Context, eventName string, authRecipe string, use
 		}
 		req.Header.Set("Content-Type", "application/json")
 
-		if webhook.Headers != nil {
-			for key, val := range webhook.Headers {
-				req.Header.Set(key, val.(string))
-			}
+		for key, val := range webhook.Headers {
+			req.Header.Set(key, val.(string))
 		}
 
 		client := &http.Client{Timeout: time.Second * 30}

Original file line number	Diff line number	Diff line change
`@@ -83,10 +83,8 @@ func RegisterEvent(ctx context.Context, eventName string, authRecipe string, use`
`83`	`83`	`}`
`84`	`84`	`req.Header.Set("Content-Type", "application/json")`
`85`	`85`
`86`		`- if webhook.Headers != nil {`
`87`		`- for key, val := range webhook.Headers {`
`88`		`- req.Header.Set(key, val.(string))`
`89`		`- }`
	`86`	`+ for key, val := range webhook.Headers {`
	`87`	`+ req.Header.Set(key, val.(string))`
`90`	`88`	`}`
`91`	`89`
`92`	`90`	`client := &http.Client{Timeout: time.Second * 30}`