diff --git a/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/QuickConnectDirectory.java b/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/QuickConnectDirectory.java index ceb8c80d9..88419440b 100644 --- a/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/QuickConnectDirectory.java +++ b/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/QuickConnectDirectory.java @@ -50,7 +50,7 @@ public class QuickConnectDirectory extends SimpleDirectory { /** * The internal counter for connection IDs. */ - private AtomicInteger connectionId; + private final AtomicInteger connectionId; /** * Creates a new QuickConnectDirectory with the default diff --git a/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/QuickConnectUserContext.java b/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/QuickConnectUserContext.java index aa98db75e..36a5a8fb0 100644 --- a/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/QuickConnectUserContext.java +++ b/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/QuickConnectUserContext.java @@ -25,9 +25,7 @@ import org.apache.guacamole.GuacamoleException; import org.apache.guacamole.net.auth.AbstractUserContext; import org.apache.guacamole.net.auth.AuthenticationProvider; import org.apache.guacamole.net.auth.ConnectionGroup; -import org.apache.guacamole.net.auth.Directory; import org.apache.guacamole.net.auth.User; -import org.apache.guacamole.net.auth.simple.SimpleDirectory; import org.apache.guacamole.net.auth.simple.SimpleUser; /** diff --git a/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/QuickConnectionGroup.java b/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/QuickConnectionGroup.java index b087c35c3..dfb60d14f 100644 --- a/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/QuickConnectionGroup.java +++ b/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/QuickConnectionGroup.java @@ -39,7 +39,7 @@ public class QuickConnectionGroup extends AbstractConnectionGroup { /** * A set that will store the Connection identifiers for this group. */ - private Set connectionIdentifiers = + private final Set connectionIdentifiers = new HashSet(Collections.emptyList()); /** diff --git a/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/rest/QuickConnectREST.java b/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/rest/QuickConnectREST.java index 559335b89..cee759141 100644 --- a/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/rest/QuickConnectREST.java +++ b/extensions/guacamole-auth-quickconnect/src/main/java/org/apache/guacamole/auth/quickconnect/rest/QuickConnectREST.java @@ -38,7 +38,7 @@ public class QuickConnectREST { /** * The connection directory for this REST endpoint. */ - private QuickConnectDirectory directory; + private final QuickConnectDirectory directory; /** * Construct a new QuickConnectREST class, taking in a diff --git a/extensions/guacamole-auth-quickconnect/src/test/java/org/apache/guacamole/auth/quickconnect/utility/QCParserTest.java b/extensions/guacamole-auth-quickconnect/src/test/java/org/apache/guacamole/auth/quickconnect/utility/QCParserTest.java index 8a4ed1b28..d63a52191 100644 --- a/extensions/guacamole-auth-quickconnect/src/test/java/org/apache/guacamole/auth/quickconnect/utility/QCParserTest.java +++ b/extensions/guacamole-auth-quickconnect/src/test/java/org/apache/guacamole/auth/quickconnect/utility/QCParserTest.java @@ -20,7 +20,6 @@ package org.apache.guacamole.auth.quickconnect.utility; import java.io.UnsupportedEncodingException; -import java.util.HashMap; import java.util.Map; import org.apache.guacamole.GuacamoleException; import org.apache.guacamole.protocol.GuacamoleConfiguration; @@ -35,6 +34,9 @@ public class QCParserTest { /** * Verify that the parseQueryString() method functions as designed. + * + * @throws UnsupportedEncodingException + * If Java lacks UTF-8 support. */ @Test public void testParseQueryString() throws UnsupportedEncodingException { @@ -51,6 +53,9 @@ public class QCParserTest { /** * Verify that the parseUserInfo() method functions as designed. + * + * @throws UnsupportedEncodingException + * If Java lacks UTF-8 support. */ @Test public void testParseUserInfo() throws UnsupportedEncodingException { @@ -79,6 +84,9 @@ public class QCParserTest { /** * Verify that the getConfiguration() method returns the expected * GuacamoleConfiguration object. + * + * @throws GuacamoleException + * If the configuration cannot be parsed from the given URI. */ @Test public void testGetConfiguration() throws GuacamoleException {