Increase events timeout to 10s, use one timeout per retry (#265)

- Change our events client timeout from 5s to 10s
- Each retry now uses a new timeout context instead of one for all of
the them

Author: Pedr0Rocha

internal/metrics/client.go

@@ -17,7 +17,7 @@ import (
 	"go.uber.org/zap"
 )
 
-const timeout = 5 * time.Second
+const timeout = 10 * time.Second
 const maxIdleConnsPerHost = 32
 const maxConnsPerHost = 32
 const maxRetries = 5
@@ -103,15 +103,15 @@ func (c *Client) RecordEvent(ctx context.Context, params EventParams) {
 	idempotency := xid.New().String()
 	logger = logger.With(zap.String("idempotency", idempotency))
 
-	// Use a new context, we want to record events of users that are already disconnected.
-	ctx, cancel := context.WithTimeout(context.Background(), timeout)
-	defer cancel()
-
 	for i := range maxRetries {
 		if i > 0 {
 			time.Sleep(time.Duration(rand.Int63n(backoffRange)*int64(i)) * time.Millisecond)
 		}
 
+		// Use a new context, we want to record events of users that are already disconnected.
+		ctx, cancel := context.WithTimeout(context.Background(), timeout)
+		defer cancel()
+
 		req, err := http.NewRequestWithContext(ctx, http.MethodPost, c.url, bytes.NewReader(payload))
 		if err != nil {
 			logger.Error("failed to create metrics request", zap.Error(err))