winstonjs / node-loggly

A client implementation for Loggly in node.js
http://github.com/winstonjs/node-loggly
Other
233 stars 81 forks source link

Remove content-length header because it messes with some proxy configs #46

Closed AshKyd closed 9 years ago

AshKyd commented 9 years ago

Submitting for your consideration, though I totally understand if it's rejected:

I can sort of guess why the content-length header it's there, but I'm just wondering how necessary it is and whether we can safely ditch it for compatibility reasons?

indexzero commented 9 years ago

Not opposed to this being an option, but it needs to be implemented in a more generic way, not just removed wholesale.