modelcontextprotocol/go-sdk
Watch
1
0
Fork
You've already forked go-sdk
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

mcp: avoid Client.Connect leaks (#1158)

Browse source 
Fixes #1154
This commit is contained in:
Guglielmo Colombo 2026-08-10 10:52:50 +02:00 • committed by GitHub
commit 4282916668
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 1 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

1
mcp/client.go
View file

@ -389,6 +389,7 @@ func (c *Client) Connect(ctx context.Context, t Transport, opts *ClientSessionOp
return nil, err return nil, err
} }
if !slices.Contains(supportedProtocolVersions, res.ProtocolVersion) { if !slices.Contains(supportedProtocolVersions, res.ProtocolVersion) {
_ = cs.Close()
return nil, unsupportedProtocolVersionError{res.ProtocolVersion} return nil, unsupportedProtocolVersionError{res.ProtocolVersion}
} }
cs.state.InitializeResult = res cs.state.InitializeResult = res