Failed to convert header keys to lower case due to field name conflict: accept-encoding about express-http-proxy HOT 6 CLOSED

Sounds like it. Can you give me a code snippet that triggers this issue, and I'll fix the bug. Thanks!

from express-http-proxy.

@monkpow , @oyvindeh I'm hitting this issue too. I cannot tell you how to reproduce it -- express (or possibly helmet which I'm using) is loading up my req.headers by some default behavior... however, i do see where the problem in express-http-proxy is happening.

This may be a suitable fix for express-http-proxy index.js line ~83:

      if (bodyEncoding(options)) {
        var acceptEncodingHeader = reqOpt.headers[ 'Accept-Encoding'.toLowerCase() ];
        if (acceptEncodingHeader) {
          reqOpt.headers[ 'Accept-Encoding'.toLowerCase() ] = acceptEncodingHeader + ', ' + bodyEncoding(options);
        } else {
          reqOpt.headers[ 'Accept-Encoding' ] = bodyEncoding(options);
        }
      }

Edit: That is short-sighted; does not take into account unique values in this header. Various ways to do this; lodash would be my toolkit, but it does not look like you are including it.

from express-http-proxy.

@oyvindeh @btmurrell I'm tentative about fixing without being able to see the problem in action. If your hunch is right, you should be able to skip this chunk of code by adding

reqBodyEnconding: null

to your options.

from express-http-proxy.

I'm sorry, I haven't had time to write a snippet that triggers the issue. Hopefully, I'll be able to do so next week.

from express-http-proxy.

I have problems reproducing this outside of the environment where I get the error (and I cannot share the full code). But I can confirm that reqBodyEncoding: null does indeed work around the issue.

from express-http-proxy.

@oyvindeh I did find a relevant bug in the code, which is patch here 861b620. This fix is in the most recent version (0.10.1) published to npm.

Let me know if you continue to have issues. Thanks for the report and clarifications so far.

from express-http-proxy.