You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@archiva.apache.org by jo...@apache.org on 2006/09/20 23:16:12 UTC

svn commit: r448344 - in /maven/archiva/trunk/archiva-webapp/src/main: resources/META-INF/plexus/application.xml webapp/WEB-INF/

Author: joakime
Date: Wed Sep 20 14:16:11 2006
New Revision: 448344

URL: http://svn.apache.org/viewvc?view=rev&rev=448344
Log:
Adding archiva.log output and audit.log output.


Modified:
    maven/archiva/trunk/archiva-webapp/src/main/resources/META-INF/plexus/application.xml
    maven/archiva/trunk/archiva-webapp/src/main/webapp/WEB-INF/   (props changed)

Modified: maven/archiva/trunk/archiva-webapp/src/main/resources/META-INF/plexus/application.xml
URL: http://svn.apache.org/viewvc/maven/archiva/trunk/archiva-webapp/src/main/resources/META-INF/plexus/application.xml?view=diff&rev=448344&r1=448343&r2=448344
==============================================================================
--- maven/archiva/trunk/archiva-webapp/src/main/resources/META-INF/plexus/application.xml (original)
+++ maven/archiva/trunk/archiva-webapp/src/main/resources/META-INF/plexus/application.xml Wed Sep 20 14:16:11 2006
@@ -26,7 +26,8 @@
 
       <configuration>
         <threshold>WARN</threshold>
-        <default-appender>console</default-appender>
+        <default-appender>console,rolling</default-appender>
+        
         <appenders>
           <appender>
             <id>console</id>
@@ -34,8 +35,58 @@
             <type>org.apache.log4j.ConsoleAppender</type>
             <conversion-pattern>%d [%t] %-5p %-30c{1} - %m%n</conversion-pattern>
           </appender>
+        
+          <appender>
+            <id>rolling</id>
+            <threshold>DEBUG</threshold>
+            <type>org.apache.log4j.DailyRollingFileAppender</type>
+            <conversion-pattern>%-4r [%t] %-5p %c %x - %m%n</conversion-pattern>
+
+            <properties>
+              <property>
+                <name>file</name>
+                <value>${plexus.home}/logs/archiva.log</value>
+              </property>
+              <property>
+                <name>append</name>
+                <value>true</value>
+              </property>
+              <property>
+                <name>datePattern</name>
+                <value>'.'yyyy-MM-dd</value>
+              </property>
+            </properties>
+          </appender>
+          
+          <appender>
+            <id>audit</id>
+            <threshold>DEBUG</threshold>
+            <type>org.apache.log4j.DailyRollingFileAppender</type>
+            <conversion-pattern>%-4r [%t] %-5p %c %x - %m%n</conversion-pattern>
+
+            <properties>
+              <property>
+                <name>file</name>
+                <value>${plexus.home}/logs/audit.log</value>
+              </property>
+              <property>
+                <name>append</name>
+                <value>true</value>
+              </property>
+              <property>
+                <name>datePattern</name>
+                <value>'.'yyyy-MM-dd</value>
+              </property>
+            </properties>
+          </appender>
         </appenders>
+        
         <levels>
+          <level>
+            <hierarchy>org.apache.maven.archiva.web.servlet.repository.RepositoryMapping</hierarchy>
+            <level>DEBUG, audit</level>
+          </level>
+        
           <!-- Help identify bugs during testing -->
           <level>
             <hierarchy>org.apache.maven</hierarchy>

Propchange: maven/archiva/trunk/archiva-webapp/src/main/webapp/WEB-INF/
------------------------------------------------------------------------------
--- svn:ignore (original)
+++ svn:ignore Wed Sep 20 14:16:11 2006
@@ -1,3 +1,5 @@
 lib
+log
+template
 classes
 database



Re: svn commit: r448344 - in /maven/archiva/trunk/archiva-webapp/src/main: resources/META-INF/plexus/application.xml webapp/WEB-INF/

Posted by Joakim Erdfelt <jo...@erdfelt.com>.
Replies are inline.

Brett Porter wrote:
>
> On 21/09/2006, at 7:16 AM, joakime@apache.org wrote:
>
>> +
>> +          <appender>
>> +            <id>audit</id>
>> +            <threshold>DEBUG</threshold>
>> +
> ...
>>          <levels>
>> +          <level>
>> +           
>> <hierarchy>org.apache.maven.archiva.web.servlet.repository.RepositoryMapping</hierarchy>
>>
>> +            <level>DEBUG, audit</level>
>> +          </level>
>
> Why DEBUG?
No reason actually.
Habit.
I believe that class only logs at info level. We could bump it up to that.
>
>> +log
>> +template
>
> Should these be cleaned too?
I chose to only ignore the log directory, as those log files could prove
useful, even in development.
>
> Also, I noticed a /template (as well as /WEB-INF/template), is that
> meant to be cleaned too? Or is ti a dupe that isn't meant to be there?
The template directory is showing up as part of a war overlay.
I haven't bothered to track down where from (yet)

- Joakim

Re: svn commit: r448344 - in /maven/archiva/trunk/archiva-webapp/src/main: resources/META-INF/plexus/application.xml webapp/WEB-INF/

Posted by Brett Porter <br...@apache.org>.
On 21/09/2006, at 7:16 AM, joakime@apache.org wrote:

> +
> +          <appender>
> +            <id>audit</id>
> +            <threshold>DEBUG</threshold>
> +
...
>          <levels>
> +          <level>
> +             
> <hierarchy>org.apache.maven.archiva.web.servlet.repository.RepositoryM 
> apping</hierarchy>
> +            <level>DEBUG, audit</level>
> +          </level>

Why DEBUG?

> +log
> +template

Should these be cleaned too?

Also, I noticed a /template (as well as /WEB-INF/template), is that  
meant to be cleaned too? Or is ti a dupe that isn't meant to be there?

- Brett


>