Maybe a Gorilla Issue... about mux HOT 7 CLOSED

Why are you handling "/" again with http.Handle?

from mux.

I have been following examples & documentation (Gorilla Mux first example):

r := mux.NewRouter()
r.HandleFunc("/", HomeHandler)
r.HandleFunc("/products", ProductsHandler)
r.HandleFunc("/articles", ArticlesHandler)
http.Handle("/", r)

How can I register the router with the http package then ?

from mux.

Yeah nevermind, it doesn't make a difference. I pass the router into ListenAndServe instead of nil.

from mux.

Ok the problem comes from line 379 in route.go in Subrouter()

r.addMatcher(router)

When you call Subrouter(), it is making a change to the route.

from mux.

Closing this for now since it's vague. If you are still having problems please file a new issue with a runnable code example.

from mux.

Hi kisielk!

I'm having this problem. You can see here a code example that other person posted on SO almost a year ago. Here you can find another example in SO.

The problem appears when we try to get another route URL form inside a Handler. For example from the PostUpdateHandler we want to redirect the user to the PostShowHandler. On those examples people (and myself) are trying to do:

rootUrl, err := mux.CurrentRoute(r).Subrouter().Get("PostShow").URL()

but this changes the original Router. So then it's when we have the problem with the 404 error.

A solution could be to use a global router variable, but I'm trying to avoid having global variables (I'm doing a clean architecture). I've tried to add the router to my Handler class but it appears to be nil on the Handler.

Any idea on how can we solve this?

thanks a lot!

from mux.

You just have to save the router or subrouter somewhere where you can access it from your route to call the URL() method. Either a global or a field on your handler struct would work. I usually just use global routers and subrouters in the majority of apps. If they are only modified at initialization time there's nothing wrong with that...

from mux.