You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by bu...@apache.org on 2018/10/30 09:35:26 UTC

[Bug 62868] New: WebappClassLoaderBase getResources(String) is not implemented

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

            Bug ID: 62868
           Summary: WebappClassLoaderBase getResources(String) is not
                    implemented
           Product: Tomcat 8
           Version: 8.5.x-trunk
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: regression
          Priority: P2
         Component: Catalina
          Assignee: dev@tomcat.apache.org
          Reporter: dkwakkel@gmail.com
  Target Milestone: ----

WebappClassLoaderBase.getResources(String) is not implemented. Resulting in
fallback to normal ClassLoader which is not self first as a webapp classloader
should be. This is a problem when e.g. using javax.activation, which uses
getResources for e.g. which mail provider will be used.

This seems a regression from
https://bz.apache.org/bugzilla/show_bug.cgi?id=3936.

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


[Bug 62868] WebappClassLoaderBase getResources(String) is not ordered according to delegate flag

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=62868

--- Comment #4 from Remy Maucherat <re...@apache.org> ---
I think that justifies keeping it open.

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


[Bug 62868] WebappClassLoaderBase getResources(String) is not ordered according to delegate flag

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=62868

Remy Maucherat <re...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |WONTFIX
             Status|NEW                         |RESOLVED
            Summary|WebappClassLoaderBase       |WebappClassLoaderBase
                   |getResources(String) is not |getResources(String) is not
                   |implemented                 |ordered according to
                   |                            |delegate flag

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


[Bug 62868] WebappClassLoaderBase getResources(String) is not ordered according to delegate flag

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=62868

Mark Thomas <ma...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #5 from Mark Thomas <ma...@apache.org> ---
Fixed in:
- trunk for 9.0.13 onwards
- 8.5.x for 8.5.35 onwards
- 7.0.x for 7.0.92 onwards

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


[Bug 62868] WebappClassLoaderBase getResources(String) is not ordered according to delegate flag

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=62868

--- Comment #2 from Remy Maucherat <re...@apache.org> ---
Although the enumeration returned has an ordering, I think the library which
starts to take into account ordering, and your trying to override a resource
(apparently) is too much to expect overall. So the issue should not be fixed
IMO.

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


[Bug 62868] WebappClassLoaderBase getResources(String) is not implemented

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=62868

Remy Maucherat <re...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|regression                  |normal

--- Comment #1 from Remy Maucherat <re...@apache.org> ---
This is not a regression at all, only the ordering is "wrong" as findResources
is (still) implemented.

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


[Bug 62868] WebappClassLoaderBase getResources(String) is not ordered according to delegate flag

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=62868

Donald <dk...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|WONTFIX                     |---
             Status|RESOLVED                    |REOPENED

--- Comment #3 from Donald <dk...@gmail.com> ---
According to the javadoc of java.lang.ClassLoader.getResources(String) the
order is important:

{quote}
@apiNote
When overriding this method it is recommended that an implementation ensures
that any delegation is consistent with the getResource(String) method. This
should ensure that the first element returned by the Enumeration's nextElement
method is the same resource that the getResource(String) method would return.
{quote}

Can you reconsider this ticket?

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