code review 6450165: net/http: add (*ServeMux).Handler method

net/http: add (*ServeMux).Handler method

The Handler method makes the ServeMux dispatch logic
available to wrappers that enforce additional constraints
on requests.

[rsc, 2012-08-20 18:55:11]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://go.googlecode.com/hg/

[bradfitzgoog, 2012-08-20 22:45:49]

LGTM

The codeSaver exists elsewhere in httptest iirc but no harm.

I assume people who care can check the type of the handler to see if it's a
redirect or not. Probably not worth returning a third result param to
indicate that.
On Aug 21, 2012 4:55 AM, <rsc@golang.org> wrote:

> Reviewers: golang-dev_googlegroups.com,
>
> Message:
> Hello golang-dev@googlegroups.com,
>
> I'd like you to review this change to
> https://go.googlecode.com/hg/
>
>
> Description:
> net/http: add (*ServeMux).Handler method
>
> The Handler method makes the ServeMux dispatch logic
> available to wrappers that enforce additional constraints
> on requests.
>
> Please review this at
http://codereview.appspot.com/**6450165/<http://codereview.appspot.com/6450165/>
>
> Affected files:
>   M src/pkg/net/http/server.go
>   A src/pkg/net/http/server_test.**go
>
>
>

[dsymonds, 2012-08-21 01:00:49]

LGTM

[rsc, 2012-08-31 16:16:36]

*** Submitted as http://code.google.com/p/go/source/detail?r=5d4729745797 ***

net/http: add (*ServeMux).Handler method

The Handler method makes the ServeMux dispatch logic
available to wrappers that enforce additional constraints
on requests.

R=golang-dev, bradfitz, dsymonds
CC=golang-dev
http://codereview.appspot.com/6450165