Uploaded image for project: 'Camel'
  1. Camel
  2. CAMEL-5047

Clean up MDC property names

        Details

        • Type: Improvement
        • Status: Resolved
        • Priority: Minor
        • Resolution: Fixed
        • Affects Version/s: 2.8.4, 2.9.0
        • Fix Version/s: 2.10.0
        • Component/s: camel-core
        • Labels:
          None
        • Patch Info:
          Patch Available
        • Estimated Complexity:
          Unknown

          Description

          Current MDC support in camel uses different property names. My proposal is to align them and use a "camel." prefix for all of them to let them live in own logical namespace:

          Before change After change
          breadcrumbId camel.breadcrumbId
          exchangeId camel.exchangeId
          messageId camel.messageId
          correlationId camel.correlationId
          routeId camel.routeId
          camelContextId camel.contextId
          transactionKey camel.transactionKey
          1. File
            CAMEL-5047.diff
            4 kB
            Łukasz Dywicki
          2. File
            CAMEL-5047-log4j.diff
            7 kB
            Łukasz Dywicki

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            splatch Łukasz Dywicki added a comment -

            Patch for issue.

            Show
            splatch Łukasz Dywicki added a comment - Patch for issue.
            Hide
            Permalink
            splatch Łukasz Dywicki added a comment -

            Patch for issue. Now with grant for ASF.

            Show
            splatch Łukasz Dywicki added a comment - Patch for issue. Now with grant for ASF.
            Hide
            Permalink
            davsclaus Claus Ibsen added a comment -

            There is some log4j.properties files which have preset the MDC keys as well. Would be nice with a patch that changes those as well.

            Show
            davsclaus Claus Ibsen added a comment - There is some log4j.properties files which have preset the MDC keys as well. Would be nice with a patch that changes those as well.
            Hide
            Permalink
            splatch Łukasz Dywicki added a comment -

            I'll find them and attach another part.

            Show
            splatch Łukasz Dywicki added a comment - I'll find them and attach another part.
            Hide
            Permalink
            davsclaus Claus Ibsen added a comment -

            How is the patch going?

            Show
            davsclaus Claus Ibsen added a comment - How is the patch going?
            Hide
            Permalink
            splatch Łukasz Dywicki added a comment -

            Log4j files affected by this issue.

            Show
            splatch Łukasz Dywicki added a comment - Log4j files affected by this issue.
            Hide
            Permalink
            davsclaus Claus Ibsen added a comment -

            Thanks for the patches.

            Show
            davsclaus Claus Ibsen added a comment - Thanks for the patches.

              People

              • Assignee:
                davsclaus Claus Ibsen
                Reporter:
                splatch Łukasz Dywicki
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development