chore: webhook add name to error msg & regenerate mock (#113)

caraml-dev · Aug 13, 2024 · 83f620d · 83f620d
1 parent c4379b4
commit 83f620d
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 3 deletions.
diff --git a/api/pkg/webhooks/client.go b/api/pkg/webhooks/client.go
@@ -90,7 +90,7 @@ func (g *simpleWebhookClient) Invoke(ctx context.Context, payload []byte) ([]byt
 			}
 			resp, err := client.Do(req)
 			if err != nil {
-				log.Errorf("Error making client request %s", err)
+				log.Errorf("Error making client request to webhook %s, err: %s", g.Name, err)
 				return err
 			}
 			defer resp.Body.Close()
@@ -103,7 +103,7 @@ func (g *simpleWebhookClient) Invoke(ctx context.Context, payload []byte) ([]byt
 			}
 			// check http status code
 			if resp.StatusCode != http.StatusOK {
-				return fmt.Errorf("response status code %d not 200, err: %s", resp.StatusCode, content)
+				return fmt.Errorf("response from %s, status code %d not 200, err: %s", g.Name, resp.StatusCode, content)
 			}
 			return nil
 

diff --git a/api/pkg/webhooks/mock_WebhookManager.go b/api/pkg/webhooks/mock_WebhookManager.go