fix(chore/ipc): change from ReadWriteCloser to ReadWriter interface

[joe0BAB]

The hijackHandler already itself takes care of closing the connection always (

secrets-engine/x/ipc/hijack.go

Line 134 in 3717adf

defer conn.Close()

). Reducing the interface passed to the callback to what actually makes sense / is allowed helps simplifying callback implementations.

[docker-agent]

Review Summary

Assessment: 🟢 APPROVE

This PR changes the hijacking callback interface from io.ReadWriteCloser to io.ReadWriter. The change is intentional and correctly implemented. The server maintains ownership of the connection lifecycle through a deferred Close() call in ServeHTTP, preventing resource leaks while still allowing callbacks to read and write. No bugs found in the changed code.