Concurrent map read and map write under Go 1.6.0 about bone HOT 10 CLOSED

map vars has been deleted In the branch beta https://github.com/go-zoo/bone/tree/beta, 11ebbcb.

from bone.

Ok cool ;)

from bone.

New error, even in the branch beta.
I got a fatal error: concurrent map writes on the muxStack[req] = m in ServeHTTP.

func (m *Mux) ServeHTTP(rw http.ResponseWriter, req *http.Request) {
    muxStack[req] = m
    // Check if a route match
    if !m.parse(rw, req) {
        // Check if it's a static ressource
        if !m.staticRoute(rw, req) {
            // Check if the request path doesn't end with /
            if !m.validate(rw, req) {
                m.HandleNotFound(rw, req)
            }
        }
    }
    delete(muxStack, req)
}

from bone.

I'm already working on this :) should be fix tomorrow. I'ts a litlle bit long, because i'm trying to find a way to drop the delay of route containing variables. Any ideas are welcome !

from bone.

Hahaha awesome :)

from bone.

It seems that it is still not working.
I still got some fatal error: concurrent map writes at the same place.

Any plans ? Because half of my services crash right now.

from bone.

It should be done tomorrow, I will notify you when it's ok. Sorry for the delay

from bone.

No problem, thanks 👍

from bone.

it should work on the beta branch now.

from bone.

Thanks a lot ;)

from bone.