Skip to content

Commit 89255c2

Browse files
Merge pull request #1108 from blublinsky/mcp-header
changed mcp authorization header to standard
2 parents e5e1454 + 99d23c8 commit 89255c2

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

internal/controller/constants.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -278,7 +278,7 @@ ssl_ca_file = '/etc/certs/cm-olspostgresca/service-ca.crt'
278278
// MCP server SSE read timeout, sec
279279
OpenShiftMCPServerHTTPReadTimeout = 30
280280
// Authorization header for OpenShift MCP server
281-
K8S_AUTH_HEADER = "kubernetes-authorization"
281+
K8S_AUTH_HEADER = "Authorization"
282282
// Constant, defining usage of kubernetes token
283283
KUBERNETES_PLACEHOLDER = "kubernetes"
284284
// MCPHeadersMountRoot is the directory hosting MCP headers in the container

internal/controller/ols_app_server_assets_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -304,7 +304,7 @@ var _ = Describe("App server assets", func() {
304304
"URL": Equal(fmt.Sprintf(OpenShiftMCPServerURL, OpenShiftMCPServerPort)),
305305
"Timeout": Equal(OpenShiftMCPServerTimeout),
306306
"SSEReadTimeout": Equal(OpenShiftMCPServerHTTPReadTimeout),
307-
"Headers": Equal(map[string]string{"kubernetes-authorization": "kubernetes"}),
307+
"Headers": Equal(map[string]string{K8S_AUTH_HEADER: KUBERNETES_PLACEHOLDER}),
308308
})),
309309
})))
310310
})
@@ -479,7 +479,7 @@ var _ = Describe("App server assets", func() {
479479
"URL": Equal(fmt.Sprintf(OpenShiftMCPServerURL, OpenShiftMCPServerPort)),
480480
"Timeout": Equal(OpenShiftMCPServerTimeout),
481481
"SSEReadTimeout": Equal(OpenShiftMCPServerHTTPReadTimeout),
482-
"Headers": Equal(map[string]string{"kubernetes-authorization": "kubernetes"}),
482+
"Headers": Equal(map[string]string{K8S_AUTH_HEADER: KUBERNETES_PLACEHOLDER}),
483483
})),
484484
})))
485485
})

0 commit comments

Comments
 (0)