x/net/http2: Transport.MaxReadFrameSize

[neild]

HTTP/2 endpoints specify a maximum frame size that they are willing to accept from their peer. The maximum frame size in each direction defaults to 16KiB, and can be set to a higher value in a SETTINGS frame.

The HTTP/2 Server.MaxReadFrameSize configuration knob allows setting the maximum frame size the server will accept. There is no corresponding knob for setting the maximum frame size the client will accept. We should add one. (We should also ensure that users don't need to set this to get good performance, of course.)

package http2
type Transport struct {
	// MaxReadFrameSize optionally specifies the largest frame
	// this transport is willing to read. A valid value is between
	// 16k and 16M, inclusive. If zero or otherwise invalid, a
	// default value is used.
	MaxReadFrameSize uint32
}

[neild]

Originating in: https://go.dev/issue/47840

[bradfitz]

Sounds fine to me.

[rsc]

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

[rsc]

Based on the discussion above, this proposal seems like a likely accept.
— rsc for the proposal review group

[rsc]

No change in consensus, so accepted. 🎉
This issue now tracks the work of implementing the proposal.
— rsc for the proposal review group

[gopherbot]

Change https://go.dev/cl/362834 mentions this issue: http2: add Transport.MaxReadFrameSize configuration setting