server: with TLS, TCP user timeout is not set on the underlying connection

[costinm]

grpc-go/internal/transport/http2_server.go

Line 235 in 60eecd9

if err = syscall.SetTCPUserTimeout(conn, kp.Timeout); err != nil {

I was looking at the write code - for net.Conn this works as expected, but if TLS is used conn will be tls.Conn so the line will
not set a timeout. I would guess rawconn should be passed ?

[dfawley]

Unassigning; I thought this was client-side, but it looks like the client is done correctly and only the server needs to be fixed. I probably don't have cycles for this right now.

[tobotg]

@zasweq @dfawley Hello!
I could look into this issue if it's still available for fix. If it's ok, please feel free to assign it to me.
Thanks

[dfawley]

Sure, that would be great if you have time for it!

[tobotg]

@dfawley I've been very busy lately, hopefully I'll be able to give it a look during the weekend.

[tobotg]

@dfawley I've just submitted a PR now. Please have a look when you get a chance. Thanks