logback date encoder pattern is incorretly using comma versus period

Summary

The logback.xml file in both activiti-admin and activiti-app are incorrectly using a comma versus a period in the encoder pattern.

• WEB-INF/classes/logback.xml

   <encoder>
        <pattern>%d{hh:mm:ss,SSS} [%t] %-5p %c %X - %m%n</pattern>
      </encoder>
  

As a result the interpretation of the timestamp is incorrect parsing SSS as timezone.

"d{pattern, timezone}" 

This is making the timestamp incorrect and confusing in logs.

Actual Behavior

• If time was "Thu Aug 30 14:04:37 EDT" the parse pattern is printing out as ex. "06:04:37"

Expected Behavior

• If time was "Thu Aug 30 14:04:37 EDT" the parse pattern should be printing out as ex. "14:04:37.593"

Workaround

• either use dot instead of comma

   <encoder>
        <pattern>%d{hh:mm:ss.SSS} [%t] %-5p %c %X - %m%n</pattern>
      </encoder>
  

or put quotes around pattern

 <encoder>
      <pattern>%d{"hh:mm:ss,SSS"} [%t] %-5p %c %X - %m%n</pattern>
    </encoder>

Additional Information

Ref:

• https://logback.qos.ch/manual/encoders.html
• https://logback.qos.ch/manual/layouts.html

  common error Given that the comma ',' character is interpreted as the parameter separator, the pattern HH:mm:ss,SSS will be interpreted as the pattern HM:mm:ss and the timezone SSS. If you wish to include a comma in your date pattern, then simply enclose the pattern between quotes. For example, %date{"HH:mm:ss,SSS"}.