code review 76320043: net/http: Request Body error should not be ignored.

net/http: Request Body error should not be ignored.

Fixes issue 7521.

[bancek, 2014-03-15 00:12:27]

Hello golang-codereviews@googlegroups.com,

I'd like you to review this change to
https://code.google.com/p/go

[bradfitz, 2014-03-17 21:42:51]

Have you filed a CLA? I don't see one for your gmail address. If so, which email
address?

[bancek, 2014-03-17 21:45:46]

On 2014/03/17 21:42:51, bradfitz wrote:
> Have you filed a CLA? I don't see one for your gmail address. If so, which
email
> address?

I've filed it as luka@bancek.net but I didn't receive any confirmation. I can
send it again with my gmail address if it will make it any easier.

[bradfitz, 2014-03-17 21:57:56]

Yes please. I see your other one filed but gmail would make it easier since
that's the email you used for this change.
On Mar 17, 2014 2:45 PM, <tr00.g33k@gmail.com> wrote:

> On 2014/03/17 21:42:51, bradfitz wrote:
>
>> Have you filed a CLA? I don't see one for your gmail address. If so,
>>
> which email
>
>> address?
>>
>
> I've filed it as luka@bancek.net but I didn't receive any confirmation.
> I can send it again with my gmail address if it will make it any easier.
>
> https://codereview.appspot.com/76320043/
>

[bancek, 2014-03-17 22:30:37]

On 2014/03/17 21:57:56, bradfitz wrote:
> Yes please. I see your other one filed but gmail would make it easier since
> that's the email you used for this change.
> On Mar 17, 2014 2:45 PM, <mailto:tr00.g33k@gmail.com> wrote:
> 
> > On 2014/03/17 21:42:51, bradfitz wrote:
> >
> >> Have you filed a CLA? I don't see one for your gmail address. If so,
> >>
> > which email
> >
> >> address?
> >>
> >
> > I've filed it as mailto:luka@bancek.net but I didn't receive any
confirmation.
> > I can send it again with my gmail address if it will make it any easier.
> >
> > https://codereview.appspot.com/76320043/
> >

Ok. Submitted.

[bradfitz, 2014-03-17 22:52:18]

LGTM

[bradfitz, 2014-03-17 22:53:22]

*** Submitted as https://code.google.com/p/go/source/detail?r=2162d6893121 ***

net/http: Request Body error should not be ignored.

Fixes issue 7521.

LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/76320043

Committer: Brad Fitzpatrick <bradfitz@golang.org>