[net] internal/http3: fix Write in Server Handler returning the wrong value

[Nicholas Husin (Gerrit)]

Nicholas Husin submitted the change

Change information

Commit message:

internal/http3: fix Write in Server Handler returning the wrong value

When changing the approach used in go.dev/cl/746760 (i.e. adjusting
slices, rather than keeping track of active part of the slices in
Write), the return value was unfortunately not updated properly.

This CL fixes the problem and modifies the test to also verify that
Write returns the correct value.

For golang/go#70914

Change-Id: Ic7dda43d47a85c74c03cf02208a0cd461b58dcf2

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

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

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

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

Files:

• M internal/http3/server.go
• M internal/http3/server_test.go

Change size: S

Delta: 2 files changed, 11 insertions(+), 4 deletions(-)

Branch: refs/heads/master

Submit Requirements:

• Code-Review: +2 by Damien Neil, +1 by Nicholas Husin
• TryBots-Pass: LUCI-TryBot-Result+1 by Go LUCI

Open in Gerrit

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

Gerrit

Gerrit-MessageType: merged

Gerrit-Project: net

Gerrit-Branch: master

Gerrit-Change-Id: Ic7dda43d47a85c74c03cf02208a0cd461b58dcf2

Gerrit-Change-Number: 748680

Gerrit-PatchSet: 2

Gerrit-Owner: Nicholas Husin <n...@golang.org>

Gerrit-Reviewer: Damien Neil <dn...@google.com>

Gerrit-Reviewer: Go LUCI <golang...@luci-project-accounts.iam.gserviceaccount.com>

Gerrit-Reviewer: Nicholas Husin <hu...@google.com>

Gerrit-Reviewer: Nicholas Husin <n...@golang.org>