add javadoc and null check for HttpScheme

Signed-off-by: Lachlan Roberts <lachlan@webtide.com>

jetty-websocket/websocket-jetty-api/src/main/java/org/eclipse/jetty/websocket/api/util/WSURI.java

@@ -103,6 +103,9 @@ public final class WSURI
     {
         Objects.requireNonNull(inputUri, "Input URI must not be null");
         String httpScheme = inputUri.getScheme();
+        if (httpScheme == null)
+            throw new URISyntaxException(inputUri.toString(), "Undefined HTTP scheme");
+
         if ("ws".equalsIgnoreCase(httpScheme) || "wss".equalsIgnoreCase(httpScheme))
             return inputUri;
 

jetty-websocket/websocket-jetty-client/src/main/java/org/eclipse/jetty/websocket/client/ClientUpgradeRequest.java

@@ -50,6 +50,10 @@ public final class ClientUpgradeRequest implements UpgradeRequest
         this.host = null;
     }
 
+    /**
+     * @deprecated use the no-args constructor instead.
+     * @see ClientUpgradeRequest#ClientUpgradeRequest()
+     */
     @Deprecated
     public ClientUpgradeRequest(URI uri)
     {