Bug 1008906

Summary:	Some serious log messages don't have id to identify them - JBoss Web
Product:	[JBoss] JBoss Enterprise Application Platform 6	Reporter:	Petr Kremensky <pkremens>
Component:	Web	Assignee:	Rémy Maucherat <rmaucher>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Nikoleta Hlavickova <nziakova>
Severity:	low	Docs Contact:
Priority:	unspecified
Version:	6.2.0	CC:	kkhan
Target Milestone:	DR1
Target Release:	EAP 6.4.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:		Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Petr Kremensky 2013-09-17 10:21:52 UTC

According to https://issues.jboss.org/browse/ANDIAMO-7 all serious log messages should have a unique code that can be used to identify them.
As a part of logging unification tests for EAP I found:
jboss-web/src/main/java/org/apache/catalina/valves/PersistentValve.java
 - 118   container.getLogger().error("deserializeError")
jboss-web/src/main/java/org/apache/catalina/session/StoreBase.java
 - 181   manager.getContainer().getLogger().error("Error getting keys", e);
jboss-web/src/main/java/org/jboss/web/php/ScriptEnvironment.java
 - 366   log.error("Invalid script names");

version: JBOSSWEB_7_2_2_FINAL

Comment 1 Rémy Maucherat 2014-01-10 15:09:59 UTC

r2337

Comment 2 Kabir Khan 2014-12-05 13:04:01 UTC

Since this appears to have been part of 6.3.0, I am setting to ON_QA for 6.4.0

Comment 3 Nikoleta Hlavickova 2014-12-09 14:30:13 UTC

There are some suspicious logs, but they all seem OK:

src/main/java/org/apache/catalina/core/ApplicationContext.java
 - 687   context.getLogger().error(message, exception);
 - 700   context.getLogger().error(message, throwable);
src/main/java/org/apache/catalina/authenticator/FormAuthenticator.java
 - 381   CatalinaLogger.AUTH_LOGGER.warn(msg, t);
 - 411   CatalinaLogger.AUTH_LOGGER.warn(msg, t);

Verified with EAP 6.4.0.DR12.