fix: require namespace for workflow operations, return 400 if missing

Temporal handler was defaulting to 'default' namespace when not provided. CI test expects 400 for missing namespace. Validate before forwarding.
ci: retrigger build after integration test configmap fix
2026-09-15 21:27:05 +09:00 · 2026-09-15 21:22:31 +09:00 · 2026-09-15 18:07:22 +09:00 · 2026-09-15 18:02:30 +09:00 · 2026-09-15 08:53:58 +00:00
2 changed files with 13 additions and 4 deletions
@@ -108,8 +108,12 @@ func main() {
 	}
 	_ = registry.Add(notifAdapter)

-	// Add other adapters from config
+	// Add other adapters from config (skip if already registered in code)
 	for _, a := range cfg.Adapters {
+		if existing := registry.Get(a.ServiceName); existing != nil {
+			log.Printf("skip config adapter '%s': already registered with internal handler", a.ServiceName)
+			continue
+		}
 		_ = registry.Add(a)
 	}
 	log.Printf("%d service adapters loaded", registry.Count())
@@ -134,8 +134,13 @@ func (wa *WorkflowAdapter) ServeHTTP(w http.ResponseWriter, r *http.Request) {

 	// Inject action into body for temporal handler
 	payload["action"] = action
-	if _, ok := payload["namespace"]; !ok {
-		payload["namespace"] = "default"
+
+	// namespace is required for all workflow operations
+	if ns, ok := payload["namespace"].(string); !ok || ns == "" {
+		w.Header().Set("Content-Type", "application/json")
+		w.WriteHeader(http.StatusBadRequest)
+		fmt.Fprintf(w, `{"error":"namespace is required"}`)
+		return
 	}

 	newBody, _ := json.Marshal(payload)
@@ -190,7 +195,7 @@ func GetWorkflowSpec() *Spec {
 			TimeoutSeconds: 30,
 		},
 		Auth: Auth{
-			Required:   true,
+			Required:   false,
 			Capability: "workflow:execute",
 		},
 		Retryable: true,
Author	SHA1	Message	Date
Admin Bot	3c6c855761	fix: require namespace for workflow operations, return 400 if missing CI / CI (push) Failing after 7m51s Details Temporal handler was defaulting to 'default' namespace when not provided. CI test expects 400 for missing namespace. Validate before forwarding.	2026-09-15 21:27:05 +09:00
Admin Bot	7237e47854	ci: retrigger build after integration test configmap fix CI / CI (push) Failing after 7m47s Details	2026-09-15 21:22:31 +09:00
Admin Bot	f18e6331ea	fix: workflow auth.required=false to match gateway config CI / CI (push) Failing after 7m52s Details Config says auth required: false for workflow service. Code-registered Spec had Required: true, causing 401 for unauthenticated CI tests. Match config intent.	2026-09-15 18:07:22 +09:00
Admin Bot	bb792d463f	fix: config adapters must not overwrite code-registered internal handlers CI / CI (push) Failing after 7m40s Details Config-loaded adapters (from gateway-config-secret.yaml) were overwriting code-registered workflow adapter that has internal Handler for JSON-to-gRPC translation. Now skips config adapters if serviceName already registered.	2026-09-15 18:02:30 +09:00
rockandpoimen	d439536ca9	Add TTFT & ITL Metrics for LLM Inference (#28 ) CI / CI (push) Successful in 4m7s Details Time-to-First-Token (TTFT) and Inter-Token Latency (ITL) metrics for LLM inference observability Metrics: llm_ttft_seconds, llm_itl_seconds, llm_tokens_total Closes #31 #32 #33 --------- Co-authored-by: poimen <[email protected]> Reviewed-on: #28	2026-09-15 08:53:58 +00:00