check `stream.headers` for options
jonathanong opened this issue · comments
the primary reason i didn't do this before because of content-encoding, but if we handle content-encoding we could just grab all necessary info from stream.headers, if available.
i'm sure they'll be edge cases if people do crazy shit...
i'm sure they'll be edge cases if people do crazy shit...
perhaps this module should just accept req instead of any stream? idk
basically i.m.o. if it looks into stream.headers, it would become more of a module specific to req, rather than the generic stream it is now. that's why i liked having the module above this one glue inflation instead of raw-body doing it, idk.
Yeah we could have something on top and put it in the express org
Yea, that would probably be... body-parser :D especially when i change it to let you build your own parser. then it's basically be your own shell to inflation + raw-body and such.
what's your opinion on this now?
Close