proxyWebSocketRequest not utilizing routes

[ghost]

While exploring node-http-proxy, among other things (such as the rather depressing documentation), I noticed that this function doesn't utilize routing preferences set when calling HttpProxy(options).

https://gist.github.com/1088665

[Marak]

I'll have someone look into this, looks like it could be a bug.

You should understand that the routing table stuff is all for convenience. If you read the "rather depressing documentation" you'll understand that building a custom routing scenario is trivial.

[indexzero]

This is easy enough to fix. For optimization purposes (i.e. less function invocations on each request), this code (https://github.com/nodejitsu/node-http-proxy/blob/master/lib/node-http-proxy.js#L344-372) should basically just be duplicated inside of proxyWebSocketRequest.

@dominctarr thoughts? Have you had a chance to really dig through the code base yet?

[dominictarr]

The gist does not have enough code to reproduce the problem, and the websocket examples are broken, but looks fairly simple, will reproduce first. the websocket examples are broken, then fix.

[dominictarr]

fixed in efa17ef

duplicating the code is a problem, will refactor when I'm more familiar with the code base.