net/http: don't panic on very large MaxBytesReaderLimit #54415
Conversation
|
This PR (HEAD: ef98b52) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/423314 to see it. Tip: You can toggle comments from me using the |
cuiweixie
force-pushed
the
http
branch
2 times, most recently
from
b5fdbbc
to
65e230f
Compare
|
This PR (HEAD: 65e230f) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/423314 to see it. Tip: You can toggle comments from me using the |
|
Message from Damien Neil: Patch Set 2: (4 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/423314. |
|
This PR (HEAD: 0675bff) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/423314 to see it. Tip: You can toggle comments from me using the |
cuiweixie
changed the title
|
This PR (HEAD: a33fe7e) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/423314 to see it. Tip: You can toggle comments from me using the |
|
Message from xie cui: Patch Set 4: (3 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/423314. |
|
Message from Damien Neil: Patch Set 4: Code-Review+2 Please don’t reply on this GitHub thread. Visit golang.org/cl/423314. |
|
Message from xie cui: Patch Set 4: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/423314. |
|
Message from xie cui: Patch Set 4: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/423314. |
|
Message from Gopher Robot: Patch Set 4: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/423314. |
|
Message from Gopher Robot: Patch Set 4: TryBot-Result+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/423314. |
|
Message from hopehook: Patch Set 4: Code-Review+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/423314. |
|
Message from David Chase: Patch Set 4: Code-Review+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/423314. |
Fixes #54408 Change-Id: I454199ae5bcd087b8fc4169b7622412105e71113 GitHub-Last-Rev: a33fe7e GitHub-Pull-Request: #54415 Reviewed-on: https://go-review.googlesource.com/c/go/+/423314 Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org> Run-TryBot: xie cui <523516579@qq.com> TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: David Chase <drchase@google.com> Reviewed-by: Damien Neil <dneil@google.com> Reviewed-by: hopehook <hopehook@qq.com>
|
This PR is being closed because golang.org/cl/423314 has been merged. |
Fixes #54408