[ 
https://issues.apache.org/jira/browse/AXIS-2780?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Andreas Veithen updated AXIS-2780:
----------------------------------

    Fix Version/s:     (was: 1.4)
                   1.4.1
    
> AxisHTTPSessionListener does not destroy ServiceLifeCycles, as it is supposed 
> to
> --------------------------------------------------------------------------------
>
>                 Key: AXIS-2780
>                 URL: https://issues.apache.org/jira/browse/AXIS-2780
>             Project: Axis
>          Issue Type: Bug
>          Components: Basic Architecture
>    Affects Versions: 1.4
>            Reporter: Olaf Krische
>             Fix For: 1.4.1
>
>
> The current code is:
>         Enumeration e = session.getAttributeNames();
>         while (e.hasMoreElements()) {
>             Object next = e.nextElement();
>             if (next instanceof ServiceLifecycle) {
>                 ((ServiceLifecycle)next).destroy();
>             }
>         }
> This code will do nothing but iterate over the "Strings" returned by 
> getAttributeNames().
> Probably you wanted to do that:
>         Enumeration e = session.getAttributeNames();
>         while (e.hasMoreElements()) {
>             String attributeName = e.nextElement().toString();
>             Object next = session.getAttribute(key);
>             if (next != null && next instanceof ServiceLifecycle) {
>                 ((ServiceLifecycle)next).destroy();
>             }
>         }
> In any way, the listener is only servlet 2.4+ compatible.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to