feat: add new metrics

snapp-incubator · Dec 15, 2024 · 4f02154 · 4f02154
1 parent c266558
commit 4f02154
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 9 deletions.
diff --git a/internal/authenticator/auto_authenticator.go b/internal/authenticator/auto_authenticator.go
@@ -53,7 +53,7 @@ func (a AutoAuthenticator) Auth(ctx context.Context, tokenString string) error {
 	if err := a.Validator.Validate(ctx, headers, "bearer "+tokenString); err != nil {
 		a.Metrics.Latency(time.Since(start).Seconds(), a.Company, err)
 
-		return fmt.Errorf("token is invalid: %w", err)
+		return fmt.Errorf("token is invalid: %w (validator response time %g)", err, time.Since(start).Seconds())
 	}
 
 	a.Metrics.Latency(time.Since(start).Seconds(), a.Company, nil)

diff --git a/pkg/validator/client.go b/pkg/validator/client.go
@@ -96,7 +96,9 @@ func (c *Client) Validate(parentCtx context.Context, headers http.Header, bearer
 		return fmt.Errorf("validator sending request failed %w", err)
 	}
 
-	closeBody(response)
+	defer func() {
+		_ = response.Body.Close()
+	}()
 
 	if response.StatusCode != http.StatusOK {
 		return ErrRequestFailed
@@ -109,10 +111,3 @@ func (c *Client) Validate(parentCtx context.Context, headers http.Header, bearer
 
 	return nil
 }
-
-// closeBody to avoid memory leak when reusing http connection.
-func closeBody(response *http.Response) {
-	if response != nil {
-		_ = response.Body.Close()
-	}
-}