Don't pass userinfo through to proxies in path position

When requesting via an HTTP proxy, we put the entire URI in the position
where usually the path and query string is expected. However, if the
original URI included a basic auth section, we must not include it, but
rather convert it to an Authorization header.

Author: mdpye

lib/em-http/http_encoding.rb

@@ -53,7 +53,12 @@ def encode_request(method, uri, query, connopts)
       # uri in request header, as opposed to a relative path.
       # Don't modify the header with CONNECT proxies. It's unneeded and will
       # cause 400 Bad Request errors with many standard setups.
-      query = uri.join(query) if connopts.proxy && !connopts.connect_proxy?
+      if connopts.proxy && !connopts.connect_proxy?
+        query = uri.join(query)
+        # Drop the userinfo, it's been converted to a header and won't be
+        # accepted by the proxy
+        query.userinfo = nil
+      end
 
       HTTP_REQUEST_HEADER % [method.to_s.upcase, query]
     end