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
Directories without a trailing slash #27 #28
Conversation
Can we get this merged and pushed to npm? |
Looks good to me. |
@pierreburgy are you saying that this is ready to be patched, or that it should be working for me in production with an npm install? |
That is ready to patch. And I look forward, to see this fix on npm too ;) |
@pierreburgy +1 |
great! |
@@ -187,6 +187,49 @@ describe('send(ctx, file)', function(){ | |||
}) | |||
}) | |||
|
|||
describe('when path does not finish with slash and format is disabled', function(){ | |||
it('should 200', function(done){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should 404
@coderhaoxin added you on npm. wanna merge and publish? |
ok! |
land bfd4cbc v2.0.0 |
Format the path to serve static file servers and not require a trailing slash for directories, so that you can do both /directory and /directory/.