[net] internal/http3: add net/http.ResponseController support

0 views

Skip to first unread message

Nicholas Husin (Gerrit)

unread,

3:36 PM (3 hours ago) 3:36 PM

to goph...@pubsubhelper.golang.org, golang-...@googlegroups.com, Nicholas Husin, Damien Neil, golang...@luci-project-accounts.iam.gserviceaccount.com, golang-co...@googlegroups.com

Nicholas Husin submitted the change

Change information

Commit message:

internal/http3: add net/http.ResponseController support

Server now implements SetReadDeadline, SetWriteDeadline,
EnableFullDuplex, and FlushError (with Hijack being left unsupported
intentionally as HTTP/3 does not support hijacking).

For golang/go#70914

Change-Id: I0821d6eec4e534e01a33f1ea923ae59d6a6a6964

Reviewed-on: https://go-review.googlesource.com/c/net/+/792000

Reviewed-by: Damien Neil <dn...@google.com>

LUCI-TryBot-Result: golang...@luci-project-accounts.iam.gserviceaccount.com <golang...@luci-project-accounts.iam.gserviceaccount.com>

Reviewed-by: Nicholas Husin <hu...@google.com>

Files:

M internal/http3/body.go
M internal/http3/conn.go
M internal/http3/roundtrip.go
M internal/http3/server.go
M internal/http3/server_test.go
M internal/http3/stream.go

Change size: L

Delta: 6 files changed, 436 insertions(+), 44 deletions(-)

Branch: refs/heads/master

Submit Requirements:

Code-Review: +2 by Damien Neil, +1 by Nicholas Husin
TryBots-Pass: LUCI-TryBot-Result+1 by golang...@luci-project-accounts.iam.gserviceaccount.com

Open in Gerrit

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

open

diffy

satisfied_requirement

Reply all

Reply to author

Forward

0 new messages