Proposal: Allow to change net/http.Server base context

[gregory-m]

Please answer these questions before submitting your issue. Thanks!

1. What version of Go are you using (go version)?
   go version go1.7beta2 darwin/amd64

2. What did you do?
   I'am trying to setup my session store and pass it to handlers via http.Request.Context()

3. What did you expect to see?
   Some way to do this.
   Like:

   ctx := context.WithValue(context.Background(), seesionStoreKey, SessionStore{})
   
   s := &http.Server{
     BaseContex: ctx
   }
   s.ListenAndServe()
   
   #And use it inside handler:
   store, ok := r.Context().Value(seesionStoreKey).(SessionStore)
   

4. What did you see instead?

I can't change base http.Request.Context()

[gopherbot]

CL https://golang.org/cl/24660 mentions this issue.

[kardianos]

I'm against this change. If you need to setup a context handler you can always have the default mux pass the request a global handler, setup the context, then pass it on to the actual mux. This also let's you specify a cancel context as well.

How the mentioned CL and example are used, it would just setup the context to be a bag of values. And the only values you could add to it would be static values; there would be no way to inject a "current" value.

[gregory-m]

Hello thank you for the response.

The problem with global handler starts if you need to initialize something only once like session store or db (I know its debatable if you want to put your db into context). Before context was moved to stdlib many people including myself was using net/context to eliminate global variables and package.Init() hell.

If we use global handler we still need global variables for all things you need to initialize once.

[kardianos]

@gregory-m Please consider the following:

// Use a closure:
mux := NewCoolMux()
db := NewPool()

http.ListenAndServe(":8080", http.HandleFunc("/", func(w, r) {
    ctx := context.Background()
    ctx = mydb.NewContext(ctx, db)
    mux.ServeHTTP(w, r.WithContext(ctx))
})

You can also use a type that implements the ServeHTTP method and has a db pool as a field. Then you can setup the context from that as well.

[gregory-m]

@kardianos clever and little bit hacky.

I don't know what core team think, but I'am good with close this issue, and see what will happen after 1.7 release and context adoption.

[pciet]

I think with Go 1 an ok application solution is a separate package with the HTTP handling, a private global context variable, and just directly call the global variable methods in your handlers.

[quentinmit]

If we were designing net/http from scratch, I would say that ListenAndServe should take a Context argument. But we can't add that now without breaking the Go1 compatibility guarantee, and I don't think it's worth adding a second ListenAndServeWithContext. :/

So a vague +1 to being able to set the context in the Server struct, even if that's not the best place for it.

[bradfitz]

@kardianos makes a good point that there are ways to do this as-is. I think I'll close this issue instead for now. I think somebody should wrap up @kardianos's example into a small func or repo somewhere. (I'd take it into https://go4.org/ too) We can revisit this in another release or so, but I imagine we don't add more API for this.

[quentinmit]

@bradfitz With @kardianos's solution, you lose the context information associated with the request by net/http. Such as the cancellation support. How would you replicate this in the handler?