Wrap handler easily

[champon1020]

Why

Because I want to use middleware in the way which is similar to net/http.

How to fix

I added method of ServeHTTP to httprouter.Handle.
Then, you can use middleware like this.

func myHandler(w http.ResponseWriter, r *http.Request, ps Params) {
    ...
    
    fmt.Println("some process")

    ...
}

func middleware(h httprouter.Handle) httprouter.Handle {
    return func(w http.ResponseWriter, r *http.Request, ps Params) {
        ...
        
        fmt.Println("some process")

        ...

        h.ServeHTTP(w, r, ps)
    }
}

func main() {
    r := httprouter.New()
    r.GET("/route", middleware(myHandler))

    ...
}

Additional

Created test.

[julienschmidt]

In general this change absolutely makes sense. Thank you!

Kindly change the godoc for ServeHTTP as written above, though.

[julienschmidt]

Please align this with the comment for http.HandlerFunc.ServeHTTP: ServeHTTP calls h(w, r, ps).

[champon1020]

Thank you for reviewing my code!
I've just fixed the comment that you have pointed out.

[fsaintjacques]

Could you merge this?

[champon1020]

@fsaintjacques I'm sorry I can't, I have no right to merge without approval.
I'm waiting for re-approval of my modification from @julienschmidt now.

[fsaintjacques]

@julienschmidt gentle ping.