common loader in catalina.properties is wrong
Affects | Status | Importance | Assigned to | Milestone | ||
---|---|---|---|---|---|---|
tomcat6 (Ubuntu) | ||||||
Xenial |
New
|
Medium
|
Unassigned | |||
tomcat7 (Debian) |
Fix Released
|
Unknown
|
||||
tomcat7 (Ubuntu) |
New
|
Medium
|
Unassigned | |||
Xenial |
New
|
Medium
|
Unassigned | |||
Yakkety |
New
|
Medium
|
Unassigned | |||
tomcat8 (Debian) |
Fix Released
|
Unknown
|
||||
tomcat8 (Ubuntu) |
Fix Released
|
Medium
|
Unassigned | |||
Xenial |
Triaged
|
Medium
|
Unassigned | |||
Yakkety |
Triaged
|
Medium
|
Unassigned |
Bug Description
[Impact]
* The order of paths in common.loader does not follow the upstream tomcat recommendations. This can lead to unexpected behavior.
[Test Case]
* The broken tomcat8 will have
common.
while the corrected version will have
common.
in catalina.
[Regression Potential]
* The primary source of regressions would be end-users relying on the old path order and thus getting a different class loaded with the 'fixed' version. However, the Ubuntu order is unspecified as being stable, and is contradictory to the public documentation.
Please fix the following line in catalina.properties in all tomcat source packages.
WRONG:
common.
CORRECT:
common.
Following problems with the wrong statement:
1. Odering is wrong: catalina.base should overrule catalina.home here (see class loader howto below).
2. catalina.home is expanded normally to /usr/share/tomcat7, but there is no common directory - it is below
/var/lib/tomcat7 (as expanded by catalina.base).
3. ${catalina.
For reference see https:/
> The locations searched by this class loader are defined by the common.loader property in
> $CATALINA_
> The default setting will search the following locations in the order they are listed:
>
> unpacked classes and resources in $CATALINA_BASE/lib
> JAR files in $CATALINA_BASE/lib
> unpacked classes and resources in $CATALINA_HOME/lib
> JAR files in $CATALINA_HOME/lib
Changed in tomcat7 (Debian): | |
status: | Unknown → New |
no longer affects: | tomcat7 |
Changed in tomcat6 (Ubuntu Yakkety): | |
status: | New → Invalid |
Changed in tomcat6 (Ubuntu): | |
status: | New → Invalid |
Changed in tomcat8 (Debian): | |
status: | Unknown → Fix Released |
Changed in tomcat8 (Ubuntu Xenial): | |
status: | New → In Progress |
Changed in tomcat8 (Ubuntu Yakkety): | |
status: | New → In Progress |
description: | updated |
Changed in tomcat8 (Ubuntu): | |
importance: | Undecided → Medium |
status: | Fix Committed → Fix Released |
Changed in tomcat8 (Ubuntu Xenial): | |
importance: | Undecided → Medium |
Changed in tomcat8 (Ubuntu Yakkety): | |
importance: | Undecided → Medium |
no longer affects: | tomcat6 (Ubuntu) |
no longer affects: | tomcat6 (Ubuntu Yakkety) |
Changed in tomcat6 (Ubuntu Xenial): | |
importance: | Undecided → Medium |
Changed in tomcat7 (Ubuntu): | |
importance: | Undecided → Medium |
Changed in tomcat7 (Ubuntu Xenial): | |
importance: | Undecided → Medium |
Changed in tomcat7 (Ubuntu Yakkety): | |
importance: | Undecided → Medium |
Changed in tomcat8 (Ubuntu Xenial): | |
status: | In Progress → Triaged |
Changed in tomcat8 (Ubuntu Yakkety): | |
status: | In Progress → Triaged |
Changed in tomcat7 (Debian): | |
status: | New → Fix Released |
Debian (and thus Zesty-Proposed which is syncing currently) is at 8.5.9-1 and has this fixed.