https://bz.apache.org/bugzilla/show_bug.cgi?id=66541

--- Comment #3 from Tom Whitmore <tom.whitm...@lyniate.com> ---
Hi, thanks very much for the quick response. Just checking up on the proposed
fix -- looks like a small issue with the delegation logic, confirmed in
debugger.

The CachedResourceURLStreamHandler is created for the wrapped URL
('associatedURL') and should delegate for this. It isn't the handler for the
'resourceURL' so won't be called for that.

I had already noticed the logic here was a bit subtle & easy to mislay -- I
propose clarifying it by renaming 'associatedURL' to 'wrappedURL' which makes
things easier to understand. 

"The Wrapped URL delegates to the Cache and the underlying Resource URL."

Patch attached.

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to