Modify the priority of :any routes, fixes #1089 #1517
Conversation
| end | ||
| subject.route :any, '*path' do | ||
| error! :not_found, 404 | ||
| end |
There was a problem hiding this comment.
This route is incorrect now.
|
@dblock I think we should add the explanation about this into UPGRADING.md, right? |
CHANGELOG.md
Outdated
| #### Fixes | ||
|
|
||
| * [#1505](https://github.com/ruby-grape/grape/pull/1505): Run `before` and `after` callbacks, but skip the rest when handling OPTIONS - [@jlfaber](https://github.com/jlfaber). | ||
| * [#1517](https://github.com/ruby-grape/grape/pull/1517): Modify the priority of ANY routes, fixes #1089 - [@namusyaka](https://github.com/namusyaka), [@wagenet](https://github.com/wagenet) |
There was a problem hiding this comment.
Missing a period at the end :) and I would do [#1517](...), [#1089]() ...:
lib/grape/router.rb
Outdated
| end | ||
|
|
||
| def self.supported_methods | ||
| @supported_methods ||= Grape::Http::Headers::SUPPORTED_METHODS + ['ANY'] |
There was a problem hiding this comment.
Maybe * instead of ANY? Just in case ANY one day becomes an HTTP verb :)
|
Definitely needs a note in UPGRADING. Thanks! |
|
@dblock Could you review this again? |
namusyaka
changed the title
CHANGELOG.md
Outdated
| #### Fixes | ||
|
|
||
| * [#1505](https://github.com/ruby-grape/grape/pull/1505): Run `before` and `after` callbacks, but skip the rest when handling OPTIONS - [@jlfaber](https://github.com/jlfaber). | ||
| * [#1517](https://github.com/ruby-grape/grape/pull/1517): Modify the priority of ANY routes, fixes #1089 - [@namusyaka](https://github.com/namusyaka), [@wagenet](https://github.com/wagenet). |
There was a problem hiding this comment.
Should be:
[#1517](https://github.com/ruby-grape/grape/pull/1517), [#1089](https://github.com/ruby-grape/grape/pull/1089): Fix: priority of ANY routes - [@namusyaka](https://github.com/namusyaka), [@wagenet](https://github.com/wagenet).
|
I corrected some English and merged via e489995. Thank you. |
|
@dblock Thanks! |
| end | ||
| end | ||
|
|
||
| send('get', 'nested') |
There was a problem hiding this comment.
Just noticing this. This should just be get 'nested/something'..., I'll update.
There was a problem hiding this comment.
Right. I missed this. Sorry ;(
ref #1089
Conventional, the :any routes had been searched after matching first route and 405 routes.
This PR fixes the issue by adding the :any routes processing before 405 processing.