0005243: TLD not loaded when compiling jsp in case of using custom class loader.

Viewing Issue Simple Details [ Jump to Notes ]

[ View Advanced ] [ Issue History ] [ Print ]

Category

Severity

Reproducibility

Date Submitted

Last Update

0005243

[Resin]

minor

always

10-10-12 15:22

01-09-13 11:44

Reporter

cowan

View Status

public

Assigned To

ferg

Priority

high

Resolution

fixed

Status

closed

Product Version

4.0.32

Summary

0005243: TLD not loaded when compiling jsp in case of using custom class loader.

Description

When setting custom ClassLoader to the ContextClassLoader of currentThread in filter,
It seems that Resin cannot resolve tld file contained
in "WEB-INF/lib" of a jar file and causes jsp compile error.

I think Resin should consider the case that parent ClassLoader is
DynamicClassLoader in the process of "com.caucho.jsp.TldManager#getClassPath(ClassLoader loader)"

The current code

{{{
private ArrayList<Path> getClassPath(ClassLoader loader)
{
   String classpath = null;

   if (loader instanceof DynamicClassLoader)
     classpath = ((DynamicClassLoader) loader).getClassPath();
   else
     classpath = CauchoSystem.getClassPath();

   return getClassPath(classpath);
}
}}}

might be

{{{
private ArrayList<Path> getClassPath(ClassLoader loader)
{
   if (loader instanceof DynamicClassLoader)
     return getClassPath(((DynamicClassLoader) loader).getClassPath());
   else if(loader.getParent() == null)
     return getClassPath(CauchoSystem.getClassPath());

   return getClassPath(loader.getParent());
}
}}}

Additional Information

Rep by S. Nobuaki

Attached Files

Relationships

Notes
(0006150) ferg 01-09-13 11:44	jsp/18jp

Issue History
Date Modified	Username	Field	Change
10-10-12 15:22	cowan	New Issue
01-09-13 11:44	ferg	Note Added: 0006150
01-09-13 11:44	ferg	Assigned To	=> ferg
01-09-13 11:44	ferg	Status	new => closed
01-09-13 11:44	ferg	Resolution	open => fixed
01-09-13 11:44	ferg	Fixed in Version	=> 4.0.34

Mantis 1.0.0rc3[^]

29 total queries executed.
26 unique queries executed.