Hi, In our internal testing we've noticed that org.apache.myfaces.config.annotation.LifecycleProviderFactory's are _never_ released. I don't think it's a problem for the default implementations, but our container integration uses a special implementation that needs to hold on to resources for the life of the LifecycleProviderFactory. Obviously over long runs we're running into memory leaks.
The fix is very simple -- just call release() in StartupServletContextListener.contextDestroyed(). The catch is that I've noticed a couple test cases where the factories are manually released, so I want to avoid doing that more than once. Is that a valid concern or do the tests just mock everything and therefore we're not even using that listener? If everyone is OK with the change I will make a new bug and fix it. Thanks, Curtiss Howard
