[CAUSEWAY-748] Improve the way we setup logging to vary between prod and test, and independent of viewer.

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: viewer-wicket-1.4.1, core-1.4.0
• Fix Version/s: 2.0.0-M3
• Component/s: Core, Viewer Wicket
• Assignee: Daniel Keir Haywood
• Reporter: Daniel Keir Haywood
• Created: 17/Mar/14 13:55
• Updated: 19/Mar/20 14:16
• Resolved: 19/Mar/20 14:16

Description

There is very similar code in both IsisWebAppBootstrapper and the IsisWicketApplication, using IsisLoggingConfigurer. This should be factored out into something reusable by both, as a context initlaizer (global webapp scope).

Then, in both cases let the logging.properties be picked up from some other directory outside of WEB-INF.

~~~
UPDATE: it seems that specifying log4j.configuration seems to override this already???

/usr/lib/jvm/java-7-openjdk-amd64/bin/java -Djava.util.logging.config.file=/var/lib/tomcat7/conf/logging.properties -Djava.awt.headless=true -Dfile.encoding=UTF-8 -server -Xms128m -Xmx768m -XX:PermSize=64m -XX:MaxPermSize=256m -XX:+DisableExplicitGC -Dlog4j.configuration=/etc/tomcat7/estatio/logging.properties -Djava.util.logging.manager=org.apache.juli.ClassLoaderLogManager -Djava.endorsed.dirs=/usr/share/tomcat7/endorsed -classpath /usr/share/tomcat7/bin/bootstrap.jar:/usr/share/tomcat7/bin/tomcat-juli.jar -Dcatalina.base=/var/lib/tomcat7 -Dcatalina.home=/usr/share/tomcat7 -Djava.io.tmpdir=/tmp/tomcat7-tomcat7-tmp org.apache.catalina.startup.Bootstrap start