Fix issue with first headers received becoming permanent.

gonzalocasas / node-proxy-middleware

proxy as middleware

MIT License

146 stars 65 forks source link

Fix issue with first headers received becoming permanent. #8

Closed MattCheely closed 11 years ago

MattCheely commented 11 years ago

Copying headers directly onto the options object passed to proxy was resulting in headers from the first request seen to clobber headers on every subsequent request. This was resolved by using a distinct options object to create the outgoing proxy request for each incoming request. I also updated header merging to not change the state of the incoming request object.

MattCheely commented 11 years ago

Nasty bug, this. Felt good to squash it.

andrewrk commented 11 years ago

Yikes that's a nasty bug. Sorry about that. Thanks for the fix.

andrewrk commented 11 years ago

bug fix landed in 0.3.1

MattCheely commented 11 years ago

No worries on the bug. Thanks for the quick merge.