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 wrote: > 2012/6/21 Dave Newton : > > Ah, okay--cool :) I don't see it in the ng filter dispatcher, but I'll > track > > it down

Re: File upload interceptor/filter dispatcher temp file cleanup

2012-06-21 Thread Łukasz Lenart
2012/6/21 Dave Newton : > 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 to Prepare.cleanupRe

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 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 > MultiPartRequest.cleanUp()

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. Regard

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 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 like to un-"ng" thi

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

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. T