net/http/httputil: fix unannounced trailers when body is empty #29032
Conversation
googlebot
added
the
cla: yes
|
This PR (HEAD: 04ca439) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/151898 to see it. Tip: You can toggle comments from me using the |
|
This PR (HEAD: fc4496b) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/151898 to see it. Tip: You can toggle comments from me using the |
|
Message from Brad Fitzpatrick: Patch Set 2: Run-TryBot+1 Code-Review+1 (5 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/151898. |
|
Message from Gobot Gobot: Patch Set 2: TryBots beginning. Status page: https://farmer.golang.org/try?commit=a667f18c Please don’t reply on this GitHub thread. Visit golang.org/cl/151898. |
|
Message from Gobot Gobot: Patch Set 2: TryBot-Result+1 TryBots are happy. Please don’t reply on this GitHub thread. Visit golang.org/cl/151898. |
|
Message from Brad Fitzpatrick: Patch Set 2: Julien, ping. Can you update this? Please don’t reply on this GitHub thread. Visit golang.org/cl/151898. |
|
This PR (HEAD: 399ae85) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/151898 to see it. Tip: You can toggle comments from me using the |
bradfitz
changed the title
|
Message from Brad Fitzpatrick: Patch Set 4: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/151898. |
|
Message from Gobot Gobot: Patch Set 4: TryBots beginning. Status page: https://farmer.golang.org/try?commit=4272176b Please don’t reply on this GitHub thread. Visit golang.org/cl/151898. |
|
This PR (HEAD: 3469adc) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/151898 to see it. Tip: You can toggle comments from me using the |
|
Message from Gobot Gobot: Patch Set 4: Build is still in progress... Consult https://build.golang.org/ to see whether it's a new failure. Other builds still in progress; subsequent failure notices suppressed until final report. Please don’t reply on this GitHub thread. Visit golang.org/cl/151898. |
|
Message from Brad Fitzpatrick: Patch Set 5: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/151898. |
|
Message from Gobot Gobot: Patch Set 5: TryBots beginning. Status page: https://farmer.golang.org/try?commit=67948164 Please don’t reply on this GitHub thread. Visit golang.org/cl/151898. |
|
Message from Gobot Gobot: Patch Set 5: Build is still in progress... Consult https://build.golang.org/ to see whether it's a new failure. Other builds still in progress; subsequent failure notices suppressed until final report. Please don’t reply on this GitHub thread. Visit golang.org/cl/151898. |
|
Message from Gobot Gobot: Patch Set 5: TryBot-Result-1 1 of 19 TryBots failed: Consult https://build.golang.org/ to see whether they are new failures. Please don’t reply on this GitHub thread. Visit golang.org/cl/151898. |
Fix unannounced trailers when body is empty and without announced trailers. Fixes #29031 Change-Id: If49951a42fe56d4be4436a999627db4c2678659d GitHub-Last-Rev: 3469adc GitHub-Pull-Request: #29032 Reviewed-on: https://go-review.googlesource.com/c/151898 Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
|
Message from Brad Fitzpatrick: Patch Set 5: Code-Review+2 Please don’t reply on this GitHub thread. Visit golang.org/cl/151898. |
|
This PR is being closed because golang.org/cl/151898 has been merged. |
Fix unannounced trailers when body is empty and without announced trailers.
Fixes #29031