MyFavMirrors/minio
1
0
Fork 0
mirror of https://github.com/minio/minio.git synced 2025-04-16 00:49:09 -04:00
Code Issues Packages Projects Releases Wiki Activity

Fix second muxclient context leak (#18987)

Browse Source 
Subrouted requests were also leaking contexts in mux clients.

Similar to #18956
This commit is contained in:
Klaus Post 2024-02-06 13:35:16 -08:00 committed by GitHub
parent 22687c1f50
commit 9bcc46d93d
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
internal/grid/connection.go
View File

@ -360,6 +360,7 @@ func (c *Subroute) Request(ctx context.Context, h HandlerID, req []byte) ([]byte
if debugReqs { if debugReqs {
fmt.Println(client.MuxID, c.String(), "Subroute.Request: DELETING MUX") fmt.Println(client.MuxID, c.String(), "Subroute.Request: DELETING MUX")
} }
client.cancelFn(context.Canceled)
c.outgoing.Delete(client.MuxID) c.outgoing.Delete(client.MuxID)
}() }()
return client.traceRoundtrip(ctx, c.trace, h, req) return client.traceRoundtrip(ctx, c.trace, h, req)