-
Notifications
You must be signed in to change notification settings - Fork 18.1k
net/http: call requestTooLarge on unwrapped ResponseWriter in MaxBytesReader #73893
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
base: master
Are you sure you want to change the base?
net/http: call requestTooLarge on unwrapped ResponseWriter in MaxBytesReader #73893
Conversation
This PR (HEAD: caba5e5) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/676676. Important tips:
|
Message from Gopher Robot: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/676676. |
Message from Gopher Robot: Patch Set 1: Congratulations on opening your first change. Thank you for your contribution! Next steps: Most changes in the Go project go through a few rounds of revision. This can be During May-July and Nov-Jan the Go project is in a code freeze, during which Please don’t reply on this GitHub thread. Visit golang.org/cl/676676. |
caba5e5
to
f7aadef
Compare
This PR (HEAD: f7aadef) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/676676. Important tips:
|
Message from Eric Gusmão: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/676676. |
Message from Damien Neil: Patch Set 3: (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/676676. |
Message from Eric Gusmão: Patch Set 3: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/676676. |
Message from Eric Gusmão: Patch Set 3: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/676676. |
This change adds an unwrapping loop to call requestTooLarge if found.
Fixes #73754