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

Response body and HEAD #1697

Merged
merged 3 commits into from Jul 6, 2015
Merged

Response body and HEAD #1697

merged 3 commits into from Jul 6, 2015

Conversation

annevk
Copy link
Member

@annevk annevk commented Mar 31, 2015

Test for w3c/ServiceWorker#655 though I'm not sure what's happening. Is the server actually writing out "HEAD" or is it doing some trickery?

@hoppipolla-critic-bot
Copy link

Critic review: https://critic.hoppipolla.co.uk/r/4453

This is an external review system which you may optionally use for the code review of your pull request.

In order to help critic track your changes, please do not make in-place history rewrites (e.g. via git rebase -i or git commit --amend) when updating this pull request.

@annevk
Copy link
Member Author

annevk commented Mar 31, 2015

This should probably not be merged as-is.

@annevk
Copy link
Member Author

annevk commented Jun 26, 2015

This is now ready for review.

@annevk
Copy link
Member Author

annevk commented Jun 26, 2015

https://critic.hoppipolla.co.uk/showcomment?chain=11603 seems to be the real issue here. The Python backend not propagating things properly.

hallvors added a commit that referenced this pull request Jul 6, 2015
Response body and HEAD - for HEAD requests drop contents even if the server sends some
@hallvors hallvors merged commit 1d10ad4 into master Jul 6, 2015
@annevk annevk deleted the xmlhttprequest-head branch July 6, 2015 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants