[go] net/http: add Server.MaxHeaderValueCount setting

0 views
Skip to first unread message

Nicholas Husin (Gerrit)

unread,
Jul 1, 2026, 11:36:01 AM (3 days ago) Jul 1
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:
net/http: add Server.MaxHeaderValueCount setting

Implement MaxHeaderValueCount support for both the HTTP/1 and HTTP/2
server. This allows servers who want to support large headers such as
SSO or OIDC cookies (and therefore has to set MaxHeaderBytes to a large
value), to still limit the number of headers that they are willing to
accept.

Also added the usual note for linknamed symbols for
net/textproto.readMIMEHeader, which seems to have been missed
originally.

Release note changes will be added directly in x/website.

For #79936
Change-Id: I600ac6c492f64f9cfa2730836be64dc76a6a6964
Reviewed-by: Nicholas Husin <hu...@google.com>
Reviewed-by: Damien Neil <dn...@google.com>
Files:
  • M api/go1.27.txt
  • M src/net/http/http2.go
  • M src/net/http/internal/http2/api.go
  • M src/net/http/internal/http2/frame.go
  • M src/net/http/internal/http2/server.go
  • M src/net/http/request.go
  • M src/net/http/serve_test.go
  • M src/net/http/server.go
  • M src/net/textproto/reader.go
Change size: M
Delta: 9 files changed, 239 insertions(+), 9 deletions(-)
Branch: refs/heads/master
Submit Requirements:
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: go
Gerrit-Branch: master
Gerrit-Change-Id: I600ac6c492f64f9cfa2730836be64dc76a6a6964
Gerrit-Change-Number: 795460
Gerrit-PatchSet: 11
Gerrit-Owner: Nicholas Husin <n...@golang.org>
Gerrit-Reviewer: Damien Neil <dn...@google.com>
Gerrit-Reviewer: Nicholas Husin <hu...@google.com>
Gerrit-Reviewer: Nicholas Husin <n...@golang.org>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages