Honor existing Origin header when using WebSockets, otherwise use secured scheme for wss, close AsyncHttpClient#1448

Motivation:

When performing initial WebSocket HTTP request, we force Origin header.

This is wrong, as Origin might use a different domain than WebSocket
url.
Also, when computing default Origin, it would make sense to use a
secure scheme when using secured sockets.

Modifications:
* Don’t override existing Origin header
* Use https for wss

Result:
It’s now possible to set Origin on a different domain. Better default

Author: slandelle

client/src/main/java/org/asynchttpclient/netty/request/NettyRequestFactory.java

@@ -194,10 +194,15 @@ public NettyRequest newNettyRequest(Request request, boolean forceConnect, Proxy
         if (!connect && uri.isWebSocket()) {
             headers.set(UPGRADE, HttpHeaderValues.WEBSOCKET)//
                     .set(CONNECTION, HttpHeaderValues.UPGRADE)//
-                    .set(ORIGIN, "http://" + uri.getHost() + ":" + uri.getExplicitPort())//
                     .set(SEC_WEBSOCKET_KEY, getKey())//
                     .set(SEC_WEBSOCKET_VERSION, "13");
 
+            if (!headers.contains(ORIGIN)) {
+                String scheme = uri.isSecured() ? "https://" : "http://";
+                String origin = scheme+ uri.getHost() + ":" + uri.getExplicitPort();
+                headers.set(ORIGIN, origin);
+            }
+
         } else if (!headers.contains(CONNECTION)) {
             CharSequence connectionHeaderValue = connectionHeader(config.isKeepAlive(), httpVersion);
             if (connectionHeaderValue != null) {