Log4j 2
  1. Log4j 2
  2. LOG4J2-475

MongoDBConnection is broken because it uses addDecodingHook instead of addEncodingHook

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-beta9
    • Fix Version/s: 2.0-rc1
    • Component/s: Appenders
    • Labels:
      None

      Description

      Seems trivial to fix, just use addEncodingHook.

      (I have a feeling MongoDB logging has never been tested! Is there a unit test for it?)

        Activity

        Tal Liron created issue -
        Hide
        Matt Sicker added a comment -

        Here's a patch.

        Show
        Matt Sicker added a comment - Here's a patch.
        Matt Sicker made changes -
        Field Original Value New Value
        Attachment 0001-Fix-LOG4J2-475.patch [ 12626593 ]
        Nick Williams made changes -
        Assignee Nick Williams [ beamerblvd ]
        Nick Williams made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Nick Williams added a comment -

        Fixed with r1565889. Based it off of Matt's patch, but there's no need to also have a decoding hook because the appender doesn't read events, it only writes them.

        Show
        Nick Williams added a comment - Fixed with r1565889. Based it off of Matt's patch, but there's no need to also have a decoding hook because the appender doesn't read events, it only writes them.
        Nick Williams made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Fix Version/s 2.0-rc1 [ 12325011 ]
        Resolution Fixed [ 1 ]
        Tal Liron made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        47d 16h 55m 1 Nick Williams 08/Feb/14 03:06
        In Progress In Progress Resolved Resolved
        6m 7s 1 Nick Williams 08/Feb/14 03:12
        Resolved Resolved Closed Closed
        52d 2h 34m 1 Tal Liron 01/Apr/14 06:46

          People

          • Assignee:
            Nick Williams
            Reporter:
            Tal Liron
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development