Fix various issues raised when adapting upper layers

[lgpl/argeo-commons.git] / org.argeo.cms.servlet / src / org / argeo / cms / servlet / ServletHttpRequest.java

diff --git a/org.argeo.cms.servlet/src/org/argeo/cms/servlet/ServletHttpRequest.java b/org.argeo.cms.servlet/src/org/argeo/cms/servlet/ServletHttpRequest.java
index 523f7b75786bbdd6b3397818745cb3672458ac21..75a0d8782c7bca2266ae76b6e010edc3825ed634 100644 (file)
--- a/org.argeo.cms.servlet/src/org/argeo/cms/servlet/ServletHttpRequest.java
+++ b/org.argeo.cms.servlet/src/org/argeo/cms/servlet/ServletHttpRequest.java
@@ -18,13 +18,12 @@ public class ServletHttpRequest implements HttpRequest {
 
        @Override
        public HttpSession getSession() {
-               return new ServletHttpSession();
+               return new ServletHttpSession(request.getSession(false));
        }
 
        @Override
        public HttpSession createSession() {
-               request.getSession(true);
-               return new ServletHttpSession();
+               return new ServletHttpSession(request.getSession(true));
        }
 
        @Override
@@ -61,23 +60,4 @@ public class ServletHttpRequest implements HttpRequest {
        public int getRemotePort() {
                return request.getRemotePort();
        }
-
-       private class ServletHttpSession implements HttpSession {
-
-               @Override
-               public boolean isValid() {
-                       try {// test http session
-                               request.getSession(false).getCreationTime();
-                               return true;
-                       } catch (IllegalStateException ise) {
-                               return false;
-                       }
-               }
-
-               @Override
-               public String getId() {
-                       return request.getSession(false).getId();
-               }
-
-       }
 }