[ 
http://jira.magnolia-cms.com/browse/MAGNOLIA-2870?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Grégory Joseph resolved MAGNOLIA-2870.
--------------------------------------

    Resolution: Fixed

Applied patch, thanks !
(with the slight change that it now *first* delegates to the servlet's destroy 
method)

> ServletDispatchingFilter is not dispatching destroy() method to wrapped 
> servlet
> -------------------------------------------------------------------------------
>
>                 Key: MAGNOLIA-2870
>                 URL: http://jira.magnolia-cms.com/browse/MAGNOLIA-2870
>             Project: Magnolia
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 4.1
>            Reporter: Erik Alphenaar
>            Assignee: Philipp Bärfuss
>            Priority: Minor
>             Fix For: 4.2
>
>         Attachments: ServletDispatchingFilter.patch
>
>   Original Estimate: 0.06d
>  Remaining Estimate: 0.06d
>
> The ServletDispatchingFilter calls the init() method of the wrapped servlet 
> on start-up.
> When the contents of the servlets tree in /server/filters/servlets are 
> changed in the magnolia admincentral, magnolia reloads the filter chain
> to enable the changes to be processed, and thus creates a new instance, and 
> calls the init() method. 
> The destroy method of the wrapped servlet does not seem to be called. 
> In my opinion it would be nice to also dispatch the destroy method to the 
> wrapped servlet. At the moment the destroy() is completely ignored
> by the ServletDispatchingFilter. This might lead to unexpected behaviour in 
> some scenarios.

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



----------------------------------------------------------------
For list details see
http://www.magnolia-cms.com/home/community/mailing-lists.html
To unsubscribe, E-mail to: <[email protected]>
----------------------------------------------------------------

Reply via email to