nvazquez commented on a change in pull request #3658: WIP: client: explicitly 
create a session handler to set session timeout
URL: https://github.com/apache/cloudstack/pull/3658#discussion_r342062476
 
 

 ##########
 File path: client/src/main/java/org/apache/cloudstack/ServerDaemon.java
 ##########
 @@ -259,14 +263,14 @@ private HandlerCollection createHandlers() {
         final RequestLogHandler log = new RequestLogHandler();
         log.setRequestLog(createRequestLog());
 
-        // Redirect root context handler
+        // Redirect root context handler_war
         MovedContextHandler rootRedirect = new MovedContextHandler();
         rootRedirect.setContextPath("/");
         rootRedirect.setNewContextURL(contextPath);
         rootRedirect.setPermanent(true);
 
         // Put rootRedirect at the end!
-        return new HandlerCollection(log, gzipHandler, rootRedirect);
+        return new Pair<>(webApp.getSessionHandler(), new 
HandlerCollection(log, gzipHandler, rootRedirect));
 
 Review comment:
   ```suggestion
           return new HandlerCollection(log, gzipHandler, rootRedirect);
   ```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to