eclecticlogic / whisper Goto Github PK

LOGGER.error("Error processing user {} request ", user);

LOGGER.error("Error processing user "+user+" request ");

package chapters.configuration;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;


public class MyApp1 {
  final static Logger logger = LoggerFactory.getLogger(MyApp1.class);

   public static void main(String[] args) {

      for(int i=0;i<10;i++){
          logger.error("NPE", new NullPointerException());
  }
  for(int i=0;i<10;i++){
    logger.error("Exiting application."+i);
  }
  }
}

<?xml version="1.0" encoding="UTF-8" ?>    
<configuration scan="true" scanPeriod="1 minute" debug="true">
    <contextName>testContext</contextName>

    <!-- Console Appender -->
    <appender name="CONSOLE" class="ch.qos.logback.core.ConsoleAppender">
        <layout class="ch.qos.logback.classic.PatternLayout">
            <pattern>%d{yyyy-MM-dd HH:mm:ss.SSS} [%X{ApplicationId}] [%X{UnitOfWorkId}] [%thread] %-5level %logger{36} - %msg%n</pattern>
        </layout>
    </appender>

    <appender name="errorEmail" class="ch.qos.logback.classic.net.SMTPAppender">
        <SMTPHost>localhost</SMTPHost>
        <SMTPPort>25</SMTPPort>

        <to>[email protected]</to>
        <from>test@localhost</from>

        <subject>%logger{20} - %m</subject>

        <layout class="ch.qos.logback.classic.PatternLayout">
            <pattern>%date %-5level %logger{35} - %message%n</pattern>
        </layout>
    </appender>

        <!-- This is the whisper appender that routes error logs to the errorEmail appender -->
    <appender name="whisper"
        class="com.eclecticlogic.whisper.logback.WhisperAppender">
        <!-- Filter out non error logs -->
        <filter class="ch.qos.logback.classic.filter.LevelFilter">
            <level>ERROR</level>
            <onMatch>ACCEPT</onMatch>
            <onMismatch>DENY</onMismatch>
        </filter>
        <!-- This is the name of the logging category to use to send out error digests. This is associated with a 
            specific smtp appender below. -->
        <digestLoggerName>digest.appender.logger</digestLoggerName>
        <!--  suppressAfter specifies the criteria to enter suppression. The example below says that if 3 errors of the same kind
        are encountered within a 5 minute window, then suppression should kick in. -->
        <suppressAfter>3 in 5 minutes</suppressAfter>
        <!-- expireAfter specifies how much of silence the logger should see for the error message being suppressed 
        before stopping suppression. --> 
        <expireAfter>4 minutes</expireAfter>
        <!-- digestFrequency specifies how often error email digests should be sent containing statistics on messages 
        suppressed -->
        <digestFrequency>10 seconds</digestFrequency>

        <!-- The pass-through appender for the normal case when suppression is not in-force. -->
        <appender-ref ref="errorEmail" />
    </appender>


    <appender name="digest" class="ch.qos.logback.core.ConsoleAppender">
        <!-- encoders are assigned the type ch.qos.logback.classic.encoder.PatternLayoutEncoder 
            by default -->
        <encoder>
            <pattern>%d{HH:mm:ss.SSS} [%thread] %-5level %logger{36} - %X{whisper.digest.subject} - %msg%n</pattern>
        </encoder>
    </appender>

    <!-- The logger name for digest is associated with a specific appender. It is important that additivity be 
         set to false otherwise this will also show up in the regular smtp appender and log file. -->
    <logger name="digest.appender.logger" level="error" additivity="false">
        <appender-ref ref="digest" />
    </logger>

    <root level="info">
        <appender-ref ref="CONSOLE" />
        <appender-ref ref="whisper" />
    </root>
</configuration>

/**
 * Record message suppression digest.
 * 
 * @param digest
 */
public void digest(final Digest digest) {
    // Write digest only if we've suppressed something. Without the null check, an error message that is written
    // just once will cause a NPE (see https://github.com/eclecticlogic/whisper/issues/1)
    final Message<E> m = this.lastMessage.get();
    if (m != null) {
        // Added check to see if any messages have been supressed since last digest otherwise
        // code sends suppression message indicating nothing happened
        if (this.messagesSinceLastDigest.get() == 0) {
            if (m.getMessageAge() > this.manager.getSuppressionExpirationTime()) {
                // Suppression ends
                this.inSuppression = false;
                this.manager.remove(this);
                this.lastMessage.set(null);
            }
        } else {
            final DigestMessage dm = new DigestMessage();
            dm.setMessage(m.getMessage());
            dm.setFullMessage(m.getFullMessage());
            dm.setMessagesSinceLastDigest(this.messagesSinceLastDigest.get());
            dm.setMessagesSinceSuppressionStart(this.messagesSinceSuppressionStart.get());
            this.messagesSinceLastDigest.set(0);
            digest.add(dm);
        }
    }
}

Exception in thread "Timer-0" java.lang.NullPointerException
    at com.eclecticlogic.whisper.core.Muffler.digest(Muffler.java:88)
    at com.eclecticlogic.whisper.core.WhisperManager.run(WhisperManager.java:122)
    at java.util.TimerThread.mainLoop(Unknown Source)
    at java.util.TimerThread.run(Unknown Source)

   } catch (SQLException ex) {
        log.log(Level.SEVERE, null, ex);
    }