feat: add tls configuration in mcpserver#114
Conversation
7907d26 to
8d093ad
Compare
Signed-off-by: dahu.kdh <dahu.kdh@alibaba-inc.com>
8d093ad to
4d8aca9
Compare
EItanya
left a comment
There was a problem hiding this comment.
So my main issue with this PR is actually the fact that this config is NOT used by the MCPServer at all, but only by the Agent which references it in kagent. I understand the reason for this as we want a consistent way to access this resource, but it feels a little bit strange. Let me think a little bit on this.
Agree. Perhaps defining the client certificate reference in the |
I tried to deploy an MCPServer. During the deployment, the kagent controller attempts to list tools from the MCP backend over HTTPS, but it fails with the following error:
To address this, I think we should enhance MCPServer by adding an HTTPTransportTLS field, so users can customize the TLS client configuration (e.g., client certificate/key and CA bundle) when connecting to the MCP server. An example: