Accept header available in context? #515
Comments
peterbourgon
added a commit
that referenced
this issue
Apr 19, 2017
Add the request Accept header to the context, under the ContextKeyRequestAccept key. Closes #515.
|
Does #516 do what you'd expect? |
|
It does, thank you for doing the work. I would've added it myself, but I didn't want to put in the "work" before I got some form of acknowledgement that this would be a good addition. Either way, thank you, and keep up the good work with this project 👍 |
yurishkuro
pushed a commit
to yurishkuro/kit
that referenced
this issue
May 17, 2017
Add the request Accept header to the context, under the ContextKeyRequestAccept key. Closes go-kit#515.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I noticed that the
PopulateRequestContextfunction does not transfer theAcceptheader from the request object to the context.I'm using a custom
ServerBeforefunction to add this header, but I was somewhat surprised, as the Accept header is something you use a lot when working with APIs that support multiple response formats.The text was updated successfully, but these errors were encountered: