File upload interceptor/filter dispatcher temp file cleanup

2012-06-21 Thread Dave Newton
Where are uploaded files currently cleaned up? The patch for WW-3490 [1] moved temp file deletion out of the interceptor into the dispatcher, but if the ng dispatcher is being used, when/how are the files deleted? I've dug a little bit and haven't seen it yet; at this point it's quicker to ask.

Re: File upload interceptor/filter dispatcher temp file cleanup

2012-06-21 Thread Dave Newton
Anyone? So far it appears as though the patch only fixed the old filter and not those of the ng variety? Also, I'm wondering if we'd like to un-ng this, or barring that, at least re-package to something named better and put a subclass in ng and deprecate. Dave On Thu, Jun 21, 2012 at 6:14 AM,

Re: File upload interceptor/filter dispatcher temp file cleanup

2012-06-21 Thread Martin Cooper
On Thu, Jun 21, 2012 at 12:05 PM, Dave Newton davelnew...@gmail.com wrote: Anyone? I am just guessing, but perhaps this? http://y.ahoo.it/Qzl2h -- Martin Cooper So far it appears as though the patch only fixed the old filter and not those of the ng variety? Also, I'm wondering if we'd

Re: File upload interceptor/filter dispatcher temp file cleanup

2012-06-21 Thread Łukasz Lenart
Hi Dave, I've changed a bit the whole cleanup idea, right now it's delegated to given implementation of MultiPartRequest, to the method cleanUp(). So a filter calls Dispatcher.cleanUpRequest(request) which delegate to MultiPartRequest.cleanUp() It works with old and with the new filters.

Re: File upload interceptor/filter dispatcher temp file cleanup

2012-06-21 Thread Dave Newton
On Thu, Jun 21, 2012 at 3:51 PM, Łukasz Lenart lukasz.len...@googlemail.com wrote: I've changed a bit the whole cleanup idea, right now it's delegated to given implementation of MultiPartRequest, to the method cleanUp(). So a filter calls Dispatcher.cleanUpRequest(request) which delegate to

Re: File upload interceptor/filter dispatcher temp file cleanup

2012-06-21 Thread Łukasz Lenart
2012/6/21 Dave Newton davelnew...@gmail.com: Ah, okay--cool :) I don't see it in the ng filter dispatcher, but I'll track it down later when I have a chance. Is there an elevator pitch for how the ng filter ends up calling it? Take a look on the finally section of doFilter(), there is a call

Re: File upload interceptor/filter dispatcher temp file cleanup

2012-06-21 Thread Dave Newton
Woohoo, I get it now... I saw the PrepareOperations before but digging w/o an IDE is a pain. Sweet--thanks again. Dave On Thu, Jun 21, 2012 at 4:11 PM, Łukasz Lenart lukasz.len...@googlemail.com wrote: 2012/6/21 Dave Newton davelnew...@gmail.com: Ah, okay--cool :) I don't see it in the ng