Note that the stray '^' is already fixed in 5.4.1

Note that the stray '^' is already fixed in 5.4.1

Ah, figured it couldn't be broken for long but I probably should have checked :)

I've rebased and remved this part.

Hello,

Thanks for your contribution, we tested your changes and we have a few issues with them. Mainly, duplicating all headers, that we know from experience that can cause issues with some web browsers.

We won't be merging your proposed changes but instead will be proposing a much more simpler Nginx example file which will pass all requests to Node directly:

location / {
    proxy_pass            http://localhost:3000;
    proxy_set_header      X-Real-IP $remote_addr;
    proxy_set_header      Host $host;
    proxy_set_header      X-Forwarded-For $proxy_add_x_forwarded_for;
    client_max_body_size  150m;

    proxy_http_version    1.1;
    proxy_set_header      Upgrade $http_upgrade;
    proxy_set_header      Connection upgrade;
} 

Thanks for your interest anyway!

Oh, so requests going to node should either add a bunch of proxy_hide_header, or the behaviour of dropping some headers from nginx was actually useful... I wonder if there's a way to explain that to gixy.. Well, won't be a problem with the new config anyway, thanks.

I'll test a basic nginx config as I don't need any performance on my end (feeling it's a bit of a shame to not at least serve blocks and the dozens of js files directly, but won't matter much to me)
Just to be clear, it basically skips all the alternative backend ports (safe origin port and ws port) and should just work anyway?