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.

Sign up for GitHub

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

Jump to bottom

incorrect version accept header does not return a 406 response #1101

Closed
wants to merge 1 commit into from
Closed

incorrect version accept header does not return a 406 response #1101

wants to merge 1 commit into from

Commits on Aug 17, 2015

  1. incorrect version accept header does not return a 406 response 

    If you've configured Grape to use accept header versioning with strict
equal to true, Grape should return a 406 response when making a request
with an incorrect version header.  However, the system was not
responding with a 406 when using the header 'Accept:application/xml'.

Adding relevant line to changelog.

Refactoring the middleware version header class:
- Breaking up the before method into more manageable pieces, removing some
of the conditional (cyclomatic) complexity
- fixing some line lengths
- removing some local variables from methods
    elliotlarson committed Aug 17, 2015
    Configuration menu
    Copy the full SHA
    8b5af58 View commit details
    Browse the repository at this point in the history