From 5c96cd63267abf184d9a689f694f0a1e4da453d5 Mon Sep 17 00:00:00 2001
From: Nick Couchman <vnick@apache.org>
Date: Sat, 24 Mar 2018 20:44:02 -0400
Subject: [PATCH] GUACAMOLE-38: Leave default port to individual protocols.

---
 .../auth/quickconnect/utility/QCParser.java          | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

diff --git a/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/utility/QCParser.java b/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/utility/QCParser.java
index 3de9e80c4..30121df46 100644
--- a/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/utility/QCParser.java
+++ b/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/utility/QCParser.java
@@ -44,11 +44,6 @@ public class QCParser {
      */
     public static final String DEFAULT_URI_HOST = "localhost";
 
-    /**
-     * The default port to use if one is not defined.
-     */
-    public static final Integer DEFAULT_URI_PORT = 22;
-
     /**
      * Parse out a URI string and get a connection from that string,
      * or an exception if the parsing fails.
@@ -89,9 +84,6 @@ public class QCParser {
         if (host == null || host.equals(""))
             host = DEFAULT_URI_HOST;
 
-        if (port == -1 || port < 1)
-            port = DEFAULT_URI_PORT;
-
         if (query != null && !query.equals(""))
             paramList = Arrays.asList(query.split("&"));
 
@@ -106,7 +98,9 @@ public class QCParser {
         GuacamoleConfiguration qcConfig = new GuacamoleConfiguration();
         qcConfig.setProtocol(protocol);
         qcConfig.setParameter("hostname",host);
-        qcConfig.setParameter("port", Integer.toString(port));
+
+        if (port > 0)
+            qcConfig.setParameter("port", Integer.toString(port));
 
         if (username != null)
             qcConfig.setParameter("username", username);