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. CAMEL-5047-log4j.diff
        7 kB
        Łukasz Dywicki
      2. CAMEL-5047.diff
        4 kB
        Łukasz Dywicki

        Activity

        Hide
        splatch Łukasz Dywicki added a comment -

        Patch for issue.

        Show
        splatch Łukasz Dywicki added a comment - Patch for issue.
        Hide
        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
        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
        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
        davsclaus Claus Ibsen added a comment -

        How is the patch going?

        Show
        davsclaus Claus Ibsen added a comment - How is the patch going?
        Hide
        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
        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