Fix redirect handler setup error

Author: ShocOne

httpclient/httpclient_client.go

@@ -122,8 +122,10 @@ func BuildClient(config ClientConfig) (*Client, error) {
 	}
 
 	// Conditionally setup redirect handling
-	redirecthandler.SetupRedirectHandler(httpClient, config.ClientOptions.FollowRedirects, config.ClientOptions.MaxRedirects, log)
-
+	if err := redirecthandler.SetupRedirectHandler(httpClient, config.ClientOptions.FollowRedirects, config.ClientOptions.MaxRedirects, log); err != nil {
+		log.Error("Failed to set up redirect handler", zap.Error(err))
+		return nil, err
+	}
 	// Create a new HTTP client with the provided configuration.
 	client := &Client{
 		APIHandler:         apiHandler,

redirecthandler/redirecthandler.go

@@ -217,10 +217,16 @@ func (r *RedirectHandler) GetRedirectHistory(req *http.Request) []*url.URL {
 }
 
 // SetupRedirectHandler configures the HTTP client for redirect handling based on the client configuration.
-func SetupRedirectHandler(client *http.Client, followRedirects bool, maxRedirects int, log logger.Logger) {
+func SetupRedirectHandler(client *http.Client, followRedirects bool, maxRedirects int, log logger.Logger) error {
 	if followRedirects {
+		if maxRedirects < 0 {
+			log.Error("Invalid maxRedirects value", zap.Int("maxRedirects", maxRedirects))
+			return fmt.Errorf("invalid maxRedirects value: %d", maxRedirects)
+		}
+
 		redirectHandler := NewRedirectHandler(log, maxRedirects)
 		redirectHandler.WithRedirectHandling(client)
 		log.Info("Redirect handling enabled", zap.Int("MaxRedirects", maxRedirects))
 	}
+	return nil
 }