Include generic error header when authentication fails.
authorMichael Jumper <zhangmaike@users.sourceforge.net>
Fri, 23 Mar 2012 07:54:02 +0000 (00:54 -0700)
committerMichael Jumper <zhangmaike@users.sourceforge.net>
Fri, 23 Mar 2012 07:54:02 +0000 (00:54 -0700)
src/main/java/net/sourceforge/guacamole/net/basic/AuthenticatingHttpServlet.java

index ba5af4a..1281714 100644 (file)
@@ -37,6 +37,9 @@ public abstract class AuthenticatingHttpServlet extends HttpServlet {
 
     private Logger logger = LoggerFactory.getLogger(AuthenticatingHttpServlet.class);
     
+    private static final String AUTH_ERROR_MESSAGE = 
+            "User not logged in or authentication failed.";
+    
     private AuthenticationProvider authProvider;
 
     @Override
@@ -84,6 +87,8 @@ public abstract class AuthenticatingHttpServlet extends HttpServlet {
             }
             catch (GuacamoleException e) {
                 logger.error("Error retrieving configuration(s) for user {}.", username);
+
+                response.setHeader("X-Guacamole-Error-Message", AUTH_ERROR_MESSAGE);
                 response.sendError(HttpServletResponse.SC_FORBIDDEN);
                 return;
             }
@@ -91,6 +96,8 @@ public abstract class AuthenticatingHttpServlet extends HttpServlet {
             if (configs == null) {
                 logger.warn("Authentication attempt from {} for user \"{}\" failed.",
                         request.getRemoteAddr(), username);
+                
+                response.setHeader("X-Guacamole-Error-Message", AUTH_ERROR_MESSAGE);
                 response.sendError(HttpServletResponse.SC_FORBIDDEN);
                 return;
             }