Log4j 2
  1. Log4j 2
  2. LOG4J2-103

Log4jLogEvent proxies ThrowableProxy objects

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-beta2
    • Fix Version/s: 2.0-beta3
    • Component/s: Core
    • Labels:

      Description

      While verifying LOG4J2-99 I stumbled across another small issue:
      When a Log4jLogEvent is reconstructed (in AsynchAppender, MapRewritePolicy or PropertiesRewritePolicy) the ThrowableProxy of the original LogEvent is used to construct the new Log4JLogEvent.
      There the ThrowableProxy is proxied again. During this the package resolution is repeated (mostly with slightly different result).
      I created a patch for Log4JLogEvent to reuse existing ThrowableProxy objects.

      Best Regards,

      Das

      1. MapRewritePolicyTest.java
        6 kB
        Das Archive
      2. Log4jLogEvent_ThrowableProxy.patch
        0.8 kB
        Das Archive

        Activity

        Hide
        Das Archive added a comment -

        Added Patch

        Show
        Das Archive added a comment - Added Patch
        Hide
        Das Archive added a comment -

        Added the Test-Class (which tests MapRewritePolicy, but found this flaw)

        Show
        Das Archive added a comment - Added the Test-Class (which tests MapRewritePolicy, but found this flaw)
        Hide
        Ralph Goers added a comment -

        Very good work! The patch was applied in revision 1403125. Please verify and close.

        Show
        Ralph Goers added a comment - Very good work! The patch was applied in revision 1403125. Please verify and close.

          People

          • Assignee:
            Ralph Goers
            Reporter:
            Das Archive
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development