On Fri, Dec 19, 2008 at 8:14 AM, Jiří Kuhn <[email protected]> wrote: > destroy() method on ftplet container is never called. I gues that something > like > > serverContext.getFtpletContainer().destroy(); > > should be placed somewhere inside DefaultFtpServer.stop() method. Probably > right just before serverContext disposition to be, let's say, consistent > with init() method. I really wonder that none has noticed it yet. Or am I > missing something?
I'm aware of this, but unsure how we best handle it. If the user has provided their own FtpletContainer (pretty unlikely but possible), I think the API client should destroy the container itself. However, in most cases, clients will simple add Ftplets to the server, in this case we should destroy on shutdown. So, should we keep track of the fact that we're using our default provided FtpletContainer and only destroy if that's true? /niklas
