Details

    • Type: Improvement
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: master
    • Fix Version/s: None
    • Component/s: Mailet Contributions
    • Labels:

      Description

      Today, one can use the LogMessage to generate log messages upon mail reception

      But the LogMessage have the following limits:

      • It do not let you configure the warn level
      • It does generate several logs. One would be simpler to review.
      • It does add some unneeded informations, like mail name already carried by the MDC
      • It does not allow logging of specific headers or specific attributeNames. This leads to missing information as well as "to much information".
      • The code quality of the mailet is poor.

      You will:

      • Ensure a single log message will be generated. For this you will call the logger one time and concatenate log messages parts.
      • Remove the log line with mail name.
      • Add a *level* configuration option. It can takes value *warn, **info, **debug* or *error*. It will be used to set the logger log level.
      • Add a *specificHeaders* configuration option. It takes a comma separated list of header names to include in the log message. By default it is empty.
      • Add a *specificAttributes* configuration option. It takes a comma separated list of attribute names to be included in the log message. By default it is empty.
      • *body* configuration option should be false by default
      • *header* configuration option should be false by default
      • *init* should propagate exception while initilizing (Integer parse exception, also passing a negative max body should throw.)
      • Remove inlined affectation for field. They can be unset on their declaration and set when init is called.

      Correct LogMessageTest accordingly.

      Don't hesitate to reach us on the gitter chat if you have any question.

        Activity

        Hide
        rafal.algo Rafał Byczek added a comment -

        Can I do this task?

        Show
        rafal.algo Rafał Byczek added a comment - Can I do this task?

          People

          • Assignee:
            Unassigned
            Reporter:
            BTellier Tellier Benoit
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:

              Development