Bug 1031448

Summary: [GSS] (6.3.0) logging-profile works for a servlet, but doesn't for a JSP
Product: [JBoss] JBoss Enterprise Application Platform 6 Reporter: Osamu Nagano <onagano>
Component: LoggingAssignee: James Perkins <jperkins>
Status: CLOSED CURRENTRELEASE QA Contact: Nikoleta Hlavickova <nziakova>
Severity: urgent Docs Contact:
Priority: unspecified    
Version: 6.1.0CC: bmaxwell, cdewolf, jperkins, kkhan, klape, myarboro
Target Milestone: DR2Flags: smumford: needinfo-
Target Release: EAP 6.4.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
In a previous release of JBoss EAP 6, logging profiles worked as expected for a servlet, but did not work for a JSP; all log messages from a JSP went to the system log context. This issue presented because the class loader that was checked was `org.apache.jasper.servlet.JasperLoader` with a parent class loader of the class loader that was registered for the log context. This issue has been resolved by adding an option to the log manager to recursively check a class loader's parent for a LogContext.
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-08-19 12:38:30 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1140417    
Bug Blocks:    
Attachments:
Description Flags
A maven project with a binary, target/logone.war. none

Description Osamu Nagano 2013-11-18 02:05:46 UTC
Description of problem:
Suppose the following logging-profile has been set in logging subsystem, and a web app has a proper entry in its MANIFEST.MF (Logging-Profile: logone). Then all messages via "com.example.logone" logger should go into a file, logone.log. It does so with a logger got in a servlet, but it doesn't work a logger got in a JSP.

--
            <logging-profiles>
                <logging-profile name="logone">
                    <file-handler name="logone">
                        <level name="INFO"/>
                        <file relative-to="jboss.server.log.dir" path="logone.log"/>
                    </file-handler>
                    <logger category="com.example.logone">
                        <level name="INFO"/>
                    </logger>
                    <root-logger>
                        <level name="INFO"/>
                        <handlers>
                            <handler name="logone"/>
                        </handlers>
                    </root-logger>
                </logging-profile>
            </logging-profiles>
--


Steps to Reproduce:
1. Set the logging-profile in the description to a standalone instance.
2. Deploy attached web app, logone/target/logone.war.
3. Access a servlet, curl http://localhost:8080/logone/HogeServlet. Its log message appears in standalone/log/logone.log, as expected.
4. Access a JSP, curl http://localhost:8080/logone/index.jsp. Its log message appears in standalone/log/server.log and CONSOLE, unexpectedly.


Actual results:
Log from the servlet goes to standalone/log/logone.log.
Log from the JSP goes to standalone/log/server.log.


Expected results:
Both log messages should go to standalone/log/logone.log.

Comment 1 Osamu Nagano 2013-11-18 02:06:58 UTC
Created attachment 825352 [details]
A maven project with a binary, target/logone.war.

Comment 2 JBoss JIRA Server 2013-11-18 02:10:34 UTC
Osamu Nagano <onagano> made a comment on jira WFLY-2524

Similar issue, but not sure if it is the same.

Comment 10 James Perkins 2014-02-19 22:36:41 UTC
Looks like the issue is the class loader being checked is a org.apache.jasper.servlet.JasperLoader with a parent class loader of the class loader that was registered for the log context. I'm not sure what the fix is going to be ATM. I mainly just wanted to note the cause.

Comment 12 Scott Mumford 2014-07-04 03:57:04 UTC
Hi James.

Could you please review the Doc Text draft I've added. Feel free to make any necessary changes as I had trouble parsing the issue.

Comment 13 Kyle Lape 2014-07-04 16:08:10 UTC
Added a slight clarification to doc text.

Comment 14 James Perkins 2014-07-07 18:29:32 UTC
This looks okay to me.

Comment 17 Nikoleta Hlavickova 2014-09-25 14:27:44 UTC
Verified with EAP 6.4.0.DR2.