dilim/containerd
1
0
Fork 0
mirror of https://github.com/containerd/containerd.git synced 2026-08-09 01:21:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix port forward error logger to not cause concurrent write

Browse Source 
Change the logging output of a struct to only list the pointer value
to remove the traversal of a map that is not thread safe and can
at times generate a concurrent map write error

Signed-off-by: Phil Estes <estesp@amazon.com>
This commit is contained in:
Ubuntu
2025-06-30 15:19:16 -04:00
committed by Phil Estes
parent 89b7202173
commit 35cbd7349e
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
internal/cri/streamingserver/portforward/httpstream.go
View File

@@ -155,7 +155,7 @@ func (h *httpStreamHandler) getStreamPair(requestID string) (*httpStreamPair, bo
func (h *httpStreamHandler) monitorStreamPair(p *httpStreamPair, timeout <-chan time.Time) {
select {
case <-timeout:
err := fmt.Errorf("(conn=%v, request=%s) timed out waiting for streams", h.conn, p.requestID)
err := fmt.Errorf("(conn=%p, request=%s) timed out waiting for streams", h.conn, p.requestID)
utilruntime.HandleError(err)
p.printError(err.Error())
case <-p.complete: