This repository has been archived by the owner. It is now read-only.

"This type of response MUST NOT have a body. Ignoring write() calls." isn't a very good warning. #2258

Closed
partheseas opened this Issue Dec 3, 2011 · 5 comments

Comments

Projects
None yet
3 participants

Can we replace this with warnings that give a little detail about the request?

Even just something like..

This type of response (HEAD) MUST NOT have a body.
This type of response (304) MUST NOT have a body.

..would be a huge improvement in my eyes.

Member

bnoordhuis commented Dec 3, 2011

Sure. Want to submit a patch?

I've never done it before, so I might mess up, but I'll definitely try.

Ok. I fixed this in tylermwashburn/node@8944e282e3319a9848a359f39a7b8cedab8df7e1. Now what should I do? :o

submit a pull request to the v0.6 branch of the main node repo.

Alright, done. Let's move to #2260 then, shall we?

@partheseas partheseas closed this Dec 4, 2011

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