Do not return a body for HEAD requests #86

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@therigu
therigu commented Sep 14, 2013

Fix from #39 with test

therigu commented Sep 14, 2013

Note, this happens when body is [] but not undef. @miyagawa, is this the correct way to specify an empty body? - it looks like it from Plack::Response

Owner

I don't think it's correct behavior for a server to change behavior response on the request method.

If you want this behavior no matter what server, use the Head middleware (Plack::Middleware::Head).

@miyagawa miyagawa closed this Sep 14, 2013
Owner

Your test passes without the patch to .pm.

therigu commented Sep 14, 2013

The test only fails intermittently, and the title of this pull request was not well chosen. Sorry for the confusion, I have updated the code and opened a new pull request #87.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment