[picocontainer-scm] [jira] Resolved: (PICO-377) PicoServletContainerFilter: HttpSession might already by invalidated after doFilter

     [ http://jira.codehaus.org/browse/PICO-377?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Paul Hammant resolved PICO-377.
-------------------------------

    Resolution: Fixed
      Assignee: Paul Hammant

applied to pico2 and pico3

> PicoServletContainerFilter: HttpSession might already by invalidated after doFilter
> -----------------------------------------------------------------------------------
>
>                 Key: PICO-377
>                 URL: http://jira.codehaus.org/browse/PICO-377
>             Project: PicoContainer
>          Issue Type: Improvement
>          Components: Pico Web Remoting
>    Affects Versions: 2.5.1
>            Reporter: Johann Burkard
>            Assignee: Paul Hammant
>            Priority: Minor
>
> I'm getting a lot of these:
> {noformat}
> Dec 13, 2010 9:47:06 AM org.apache.catalina.core.StandardWrapperValve invoke
> SEVERE: Servlet.service() for servlet UploadServlet threw exception
> java.lang.IllegalStateException: setAttribute: Session already invalidated
> 	at org.apache.catalina.session.StandardSession.setAttribute(StandardSession.java:1326)
> 	at org.apache.catalina.session.StandardSession.setAttribute(StandardSession.java:1291)
> 	at org.apache.catalina.session.StandardSessionFacade.setAttribute(StandardSessionFacade.java:130)
> 	at org.picocontainer.web.PicoServletContainerFilter.doFilter(PicoServletContainerFilter.java:135)
> {noformat}
> In PicoServletContainerFilter, this
> {code}
>         if (!isStateless) {
>             if (printSessionSize) {
>                 PrintSessionSizeDetailsForDebugging.printItIfDebug(debug, ssh);
>             }
>             sess.setAttribute(SessionStoreHolder.class.getName(), ssh);
>         }
> {code}
> should probably be
> {code}
>         if (!isStateless) {
>             if (printSessionSize) {
>                 PrintSessionSizeDetailsForDebugging.printItIfDebug(debug, ssh);
>             }
>             try {
>                 sess.setAttribute(SessionStoreHolder.class.getName(), ssh);
>             }
>             catch (IllegalStateException ex) { /* Shit */ }
>         }
> {code}

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

<hr/>
<p>
To unsubscribe from this list please visit:
</p>
<p>
    <a href="http://xircles.codehaus.org/manage_email">http://xircles.codehaus.org/manage_email</a>