-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[#498] Dry up OPTIONS regression #573
Comments
Could you please write a test for this in Grape proper? Appreciate it. |
cc: @karlfreeman |
@dblock I added failing spec, but I need feedback where the proper place for it and how to name it. |
I think its location is fine. Now can you fix it? :) |
Thought I'd chime in on this. Giving this a quick look at, I'm able to reproduce the error with this test karlfreeman@109d4e3 (eg the first test passes, whereas the second doesn't). It appears that using path versioning with a mounted api fails. I'm not going to be able to commit the time to fix this but figured I'd see if it was something simple. |
This pull request broke
OPTIONS
for endpoints mounted inside root app withversion
.Link to demo app
The text was updated successfully, but these errors were encountered: