Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check AcceptHeaderVersioning with content negotiation in place #2868

Merged
merged 1 commit into from Apr 24, 2015

Conversation

ticosax
Copy link
Contributor

@ticosax ticosax commented Apr 23, 2015

When request contains Accept header with value */*; version=1 then the request.accepted_media_type attribute looses additional parameters send to Accept header.
Thus defeating versioning support.

@tomchristie
Copy link
Member

tomchristie commented Apr 24, 2015

Seems reasonable enough, sure.

@tomchristie tomchristie added this to the 3.1.2 Release milestone Apr 24, 2015
tomchristie added a commit that referenced this pull request Apr 24, 2015
Check AcceptHeaderVersioning with content negotiation in place
@tomchristie tomchristie merged commit 450c541 into encode:master Apr 24, 2015
1 check passed
@ticosax ticosax deleted the versioning-header-all branch Apr 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants