Uploaded image for project: 'Log4j 2'
  1. Log4j 2
  2. LOG4J2-1838

Add support for appending common suffix to each line of throwable stack trace

    Details

    • Type: New Feature
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.8.1
    • Fix Version/s: 2.8.2
    • Component/s: Pattern Converters
    • Labels:

      Description

      Add support for appending common suffix to each line of throwable stack trace. The suffix can be configured by patterns just like patterns used elsewhere.

      For example, pattern %xEx{suffix(- %mdc{key})} will append a string generated by pattern - and %mdc{key}.

      But patterns that handles throwable will be automatically ignored, because they will make the rendered result confusing. So pattern %xEx{suffix(- %rEx%mdc{key})} has the same effect with %xEx{suffix(- %mdc{key})}, as if the nested %rEx pattern has never appeared.

      Following is an example of the with such suffix:

      stack trace, with pattern "%xEx{suffix(- %mdc{key})}" and MDC(key)="test mdc suffix"
      java.lang.IllegalArgumentException: IllegalArgument - test mdc suffix
      	at org.apache.logging.log4j.core.pattern.ExtendedThrowablePatternConverterTest.testSuffixFromNormalPattern(ExtendedThrowablePatternConverterTest.java:48) [test-classes/:?] - test mdc suffix
      	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_91] - test mdc suffix
      	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:1.8.0_91] - test mdc suffix
      	at java.lang.reflect.Method.invoke(Method.java:498) ~[?:1.8.0_91] - test mdc suffix
      	at com.intellij.rt.execution.application.AppMain.main(AppMain.java:147) [idea_rt.jar:?] - test mdc suffix
      Caused by: java.lang.NullPointerException: null pointer - test mdc suffix
      	at org.apache.logging.log4j.core.pattern.ExtendedThrowablePatternConverterTest.testSuffixFromNormalPattern(ExtendedThrowablePatternConverterTest.java:47) ~[test-classes/:?] - test mdc suffix
      	... 27 more - test mdc suffix
      }}
      

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                jvz Matt Sicker
                Reporter:
                xnslong Zilong Song
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: