Remove use of X-Guacamole-Error-Message header, return null rather than throwing...

diff --git a/src/main/java/net/sourceforge/guacamole/net/basic/AuthenticatingHttpServlet.java b/src/main/java/net/sourceforge/guacamole/net/basic/AuthenticatingHttpServlet.java
index b2aa8ff..e696986 100644 (file)
--- a/src/main/java/net/sourceforge/guacamole/net/basic/AuthenticatingHttpServlet.java
+++ b/src/main/java/net/sourceforge/guacamole/net/basic/AuthenticatingHttpServlet.java
@@ -150,7 +150,6 @@ public abstract class AuthenticatingHttpServlet extends HttpServlet {
      * @throws IOException If an error occurs while sending the error.
      */
     private void failAuthentication(HttpServletResponse response) throws IOException {
      * @throws IOException If an error occurs while sending the error.
      */
     private void failAuthentication(HttpServletResponse response) throws IOException {

-        response.setHeader("X-Guacamole-Error-Message", AUTH_ERROR_MESSAGE);

         response.sendError(HttpServletResponse.SC_FORBIDDEN);
     }
     
         response.sendError(HttpServletResponse.SC_FORBIDDEN);
     }
     

diff --git a/src/main/java/net/sourceforge/guacamole/net/basic/BasicGuacamoleTunnelServlet.java b/src/main/java/net/sourceforge/guacamole/net/basic/BasicGuacamoleTunnelServlet.java
index 2962e50..a04f5e7 100644 (file)
--- a/src/main/java/net/sourceforge/guacamole/net/basic/BasicGuacamoleTunnelServlet.java
+++ b/src/main/java/net/sourceforge/guacamole/net/basic/BasicGuacamoleTunnelServlet.java
@@ -176,8 +176,8 @@ public class BasicGuacamoleTunnelServlet extends AuthenticatingHttpServlet {
             // Get authorized config
             GuacamoleConfiguration config = configs.get(id);
             if (config == null) {
             // Get authorized config
             GuacamoleConfiguration config = configs.get(id);
             if (config == null) {

-                logger.error("Error retrieving authorized configuration id={}.", id);
-                throw new GuacamoleException("Unknown configuration ID.");
+                logger.error("Configuration id={} not found.", id);
+                return null;

             }
             
             logger.info("Successful connection from {} to \"{}\".", request.getRemoteAddr(), id);
             }
             
             logger.info("Successful connection from {} to \"{}\".", request.getRemoteAddr(), id);