fix: correct URL parameter separator from ',' to '&' in connection endpoint (#19)

0x5457 · web-flow · commit 19c377bd7e28 · 2024-12-18T20:03:43.000+08:00
diff --git a/internal/webhook/v1/pod_webhook.go b/internal/webhook/v1/pod_webhook.go
@@ -199,7 +199,7 @@ func (m *TensorFusionPodMutator) patchTFClient(pod *corev1.Pod, tfReq []TFReq) (
 				})
 				container.Env = append(container.Env, corev1.EnvVar{
 					Name:  constants.GetConnectionURLEnv,
-					Value: fmt.Sprintf("%s/api/connection?name=%s,namespace=%s", m.Config.OperatorEndpoint, connectionName, connectionNamespace),
+					Value: fmt.Sprintf("%s/api/connection?name=%s&namespace=%s", m.Config.OperatorEndpoint, connectionName, connectionNamespace),
 				})
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -199,7 +199,7 @@ func (m TensorFusionPodMutator) patchTFClient(pod corev1.Pod, tfReq []TFReq) (`
`199`	`199`	`})`
`200`	`200`	`container.Env = append(container.Env, corev1.EnvVar{`
`201`	`201`	`Name: constants.GetConnectionURLEnv,`
`202`		`- Value: fmt.Sprintf("%s/api/connection?name=%s,namespace=%s", m.Config.OperatorEndpoint, connectionName, connectionNamespace),`
	`202`	`+ Value: fmt.Sprintf("%s/api/connection?name=%s&namespace=%s", m.Config.OperatorEndpoint, connectionName, connectionNamespace),`
`203`	`203`	`})`
`204`	`204`	`}`
`205`	`205`	`}`