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

--- Comment #3 from Tom Whitmore <[email protected]> ---
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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to