search

gonzalocasas / node-proxy-middleware

proxy as middleware
MIT License
146 stars 65 forks source link

Added support for via header #10

Closed stevewillcock closed 11 years ago

stevewillcock commented 11 years ago

This adds via header support to resolve #4. I thought it was safest to make this opt-in to avoid breaking any existing implementations where the via header may cause an issue for whatever reason - e.g. I noticed that you had removed the host header because of a dotcloud issue and I wanted to avoid any similar problems. It might be more 'correct' to make it opt out, as the W3C 'via' spec says that this header is mandatory for gateways and proxies. Let me know if you want it opt-out instead and I can change it.

andrewrk commented 11 years ago

Looks great, thanks! Published as 0.4.0.