vy commented on issue #3885:
URL: 
https://github.com/apache/logging-log4j2/issues/3885#issuecomment-3195536654

   ```
   [Compatibility failed for pattern: 'ISO8601_OFFSET_DATE_TIME_HH']
   expected: "2025-08-17T12:10:08,887+0530"
    but was: "2025-08-17T12:10:08,887+05"
   ```
   
   `NamedInstantPattern.ISO8601_OFFSET_DATE_TIME_HH` is defined as follows:
   
   ```
   ISO8601_OFFSET_DATE_TIME_HH("yyyy-MM-dd'T'HH:mm:ss,SSSx", 
"yyyy-MM-dd'T'HH:mm:ss,SSSX")
   ```
   
   Note the 2nd argument denotes the legacy pattern.
   `FixedDateFormat`, delegated from `InstantPatternLegacyFormatter`, contains 
the following:
   
   ```
           /**
            * ISO8601 time format: {@code "yyyy-MM-dd'T'HH:mm:ss,SSSX"} with a 
time zone like {@code -07}.
            */
           ISO8601_OFFSET_DATE_TIME_HH(
                   "yyyy-MM-dd'T'HH:mm:ss,SSSX", "yyyy-MM-dd'T'", 2, ':', 1, 
',', 1, 3, FixedTimeZoneFormat.HH),
   ```
   
   This certainly does not work, since `FixedTimeZoneFormat.HH` only encodes 
hours, whereas in our case, the time zone contains minutes component. Put 
another way, this is a bug in the **legacy** `FixedDateFormat`.
   
   I'm not inclined to fix this – we deprecated legacy formatters for a reason.
   
   @ppkarwasz, unless you've objections, I'd like to close this issue as `Not 
planned`.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@logging.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to