code review 132750043: net/http/httputil: Pass a Logger to ReverseProxy, allow...

net/http/httputil: Pass a Logger to ReverseProxy, allowing the user to control logging.

Fixes issue 8553.

[mark.theunissen, 2014-08-21 10:08:54]

Hello golang-codereviews@googlegroups.com,

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

[dave_cheney.net, 2014-08-21 10:17:01]

On 2014/08/21 10:08:54, mark.theunissen wrote:
> Hello mailto:golang-codereviews@googlegroups.com,
> 
> I'd like you to review this change to
> https://code.google.com/p/go

Please remove the patch to CONTRIBUTORS, hg file 132750043 CONTRIBUTORS

When you proposal is ready to land a committer will add you to the CONTRIBUTORS
and AUTHORS files as a separate commit.

[mark.theunissen, 2014-08-21 10:41:57]

Hello golang-codereviews@googlegroups.com, dave@cheney.net (cc:
golang-codereviews@googlegroups.com),

Please take another look.

[mark.theunissen, 2014-08-21 10:42:48]

On 2014/08/21 10:17:01, dfc wrote:
> On 2014/08/21 10:08:54, mark.theunissen wrote:
> > Hello mailto:golang-codereviews@googlegroups.com,
> > 
> > I'd like you to review this change to
> > https://code.google.com/p/go
> 
> Please remove the patch to CONTRIBUTORS, hg file 132750043 CONTRIBUTORS
> 
> When you proposal is ready to land a committer will add you to the
CONTRIBUTORS
> and AUTHORS files as a separate commit.


Sorry, first time contributor here. :)

[bradfitz, 2014-08-21 17:15:23]

LGTM

[bradfitz, 2014-08-21 17:16:39]

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

net/http/httputil: Pass a Logger to ReverseProxy, allowing the user to control
logging.

Fixes issue 8553.

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

Committer: Brad Fitzpatrick <bradfitz@golang.org>