Details
Description
Looking into the source of SessionDispatcher, I found in
122 try {
123 session.invalidate();
124 } catch (IllegalStateException e) {
125 Log.info("Session already invalidated.");
126 } finally {
127 SessionIDs.remove(session.getId());
128 }
I think line 127 causes another exception because, the session was invalidated in line 123 and some jsp-containers (at least oc4j) throws an exception when trying to get the invalidated session-id with session.getId(). Is this a potential bug?
122 try {
123 session.invalidate();
124 } catch (IllegalStateException e) {
125 Log.info("Session already invalidated.");
126 } finally {
127 SessionIDs.remove(session.getId());
128 }
I think line 127 causes another exception because, the session was invalidated in line 123 and some jsp-containers (at least oc4j) throws an exception when trying to get the invalidated session-id with session.getId(). Is this a potential bug?
Activity
Ken Fyten
created issue -
Ken Fyten
made changes -
Field | Original Value | New Value |
---|---|---|
ICEfaces Forum Reference | http://www.icefaces.org/JForum/posts/list/8482 | http://www.icefaces.org/JForum/posts/list/8482.page#35404 |
Fix Version/s | 1.7.1 [ 10122 ] | |
Assignee Priority | P2 | |
Assignee | Mircea Toma [ mircea.toma ] |
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #16733 | Tue May 27 14:32:28 MDT 2008 | mircea.toma | |
Files Changed | ||||
MODIFY
/icefaces/trunk/icefaces/core/src/com/icesoft/faces/webapp/http/servlet/SessionDispatcher.java
|
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #16734 | Tue May 27 14:33:33 MDT 2008 | mircea.toma | |
Files Changed | ||||
MODIFY
/icefaces/branches/icefaces-1.7/icefaces/core/src/com/icesoft/faces/webapp/http/servlet/SessionDispatcher.java
|
Mircea Toma
made changes -
Status | Open [ 1 ] | Resolved [ 5 ] |
Resolution | Fixed [ 1 ] |
Ken Fyten
made changes -
Status | Resolved [ 5 ] | Closed [ 6 ] |
Assignee Priority | P2 | |
Assignee | Mircea Toma [ mircea.toma ] |
Copy session ID into a variable before invalidating the session.