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

Missing POST route does not cause an error #1

Closed
aalmiray opened this issue Mar 18, 2013 · 1 comment
Closed

Missing POST route does not cause an error #1

aalmiray opened this issue Mar 18, 2013 · 1 comment
Assignees
Milestone

Comments

@aalmiray
Copy link
Contributor

Following Tim's recent Ratpack presentation at 33rd Degree

https://github.com/tlberglund/33rd-degree-ratpack

At one point the application had a GET route defined as

get('/lookup') {
    render('artist.html')
}

The template artists.html had a form with POST as the method. Clicking on the button produced an empty page. There was no error on the console. I believe this case should cause and error as there is no exact route defined for POST('/lookup') and apparently GET('/lookup') did not handle the request either.

ldaley added a commit that referenced this issue Apr 12, 2013
@ldaley
Copy link
Member

ldaley commented Apr 12, 2013

It does do the right thing from a HTTP point of view in that it returns a 405 response. What's missing though is the a way to have pretty HTTP response code pages.

Fancy contributing? :)

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

No branches or pull requests

2 participants