fix(automl,autorag): log port-forward errors instead of silently dropping

Address review feedback: ForwardURL errors in the S3 handler and
LlamaStack middleware were silently swallowed. Now log warnings
consistent with the pattern used in AttachPipelineServerClient.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: chrjones-rh

packages/automl/bff/internal/api/s3_handler.go

@@ -149,7 +149,10 @@ func (app *App) resolveS3Client(w http.ResponseWriter, r *http.Request, secretNa
 	// Dev-only: rewrite S3 endpoint to localhost via dynamic port-forward.
 	// portForwardManager is nil in production (requires DevMode=true).
 	if app.portForwardManager != nil && creds.EndpointURL != "" {
-		if rewritten, pfErr := app.portForwardManager.ForwardURL(ctx, creds.EndpointURL); pfErr == nil {
+		if rewritten, pfErr := app.portForwardManager.ForwardURL(ctx, creds.EndpointURL); pfErr != nil {
+			app.logger.Warn("dynamic port-forward failed for S3 endpoint, using original URL",
+				"error", pfErr, "url", creds.EndpointURL)
+		} else {
 			creds.EndpointURL = rewritten
 		}
 	}

packages/autorag/bff/internal/api/middleware.go

@@ -387,7 +387,10 @@ func (app *App) AttachLlamaStackClientFromSecret(next func(http.ResponseWriter,
 			// Dev-only: rewrite LlamaStack URL to localhost via dynamic port-forward.
 			// portForwardManager is nil in production (requires DevMode=true).
 			if app.portForwardManager != nil {
-				if rewritten, pfErr := app.portForwardManager.ForwardURL(ctx, baseURL); pfErr == nil {
+				if rewritten, pfErr := app.portForwardManager.ForwardURL(ctx, baseURL); pfErr != nil {
+					logger.Warn("dynamic port-forward failed for LlamaStack endpoint, using original URL",
+						"error", pfErr, "url", baseURL)
+				} else {
 					baseURL = rewritten
 				}
 			}

packages/autorag/bff/internal/api/s3_handler.go

@@ -147,7 +147,10 @@ func (app *App) resolveS3Client(w http.ResponseWriter, r *http.Request, secretNa
 	// Dev-only: rewrite S3 endpoint to localhost via dynamic port-forward.
 	// portForwardManager is nil in production (requires DevMode=true).
 	if app.portForwardManager != nil && creds.EndpointURL != "" {
-		if rewritten, pfErr := app.portForwardManager.ForwardURL(ctx, creds.EndpointURL); pfErr == nil {
+		if rewritten, pfErr := app.portForwardManager.ForwardURL(ctx, creds.EndpointURL); pfErr != nil {
+			app.logger.Warn("dynamic port-forward failed for S3 endpoint, using original URL",
+				"error", pfErr, "url", creds.EndpointURL)
+		} else {
 			creds.EndpointURL = rewritten
 		}
 	}