do not modify the query string

[richardkazuomiller]

Fix the issue of multiple consecutive slashes being replaced by a single slash referenced in #703 and #710 by separating the query string from the rest of the URL in common.urlJoin. I tried removing trailing slashes as suggested by @yulesyu in #703 but that still caused issues if the query string ended in //

[indexzero]

Except for minor style things this lgtm @jcrugzz. It even has a new test 👍

[jcrugzz]

@richardkazuomiller this is a good start. I think this function still needs some more love to handle the other edge cases but thank you very much! I'll merge and fix the stylings. Just not that we like less var statements ;)

[richardkazuomiller]

Thanks! I'm a long time node-http-proxy user and happy to finally be able to contribute. I checked your last commit I'll keep the var thing in mind for next time (^_^)

[indexzero]

User becomes contributor 👍 💯