ppkarwasz commented on issue #4024: URL: https://github.com/apache/logging-log4j2/issues/4024#issuecomment-4561921406
Hi @mjrosenb, Sorry for the long silence: this one slipped through. The bug is real: [`PropertiesConfigurationBuilder#createLogger`](https://github.com/apache/logging-log4j2/blob/rel/2.25.3/log4j-core/src/main/java/org/apache/logging/log4j/core/config/properties/PropertiesConfigurationBuilder.java#L272) extracts the known logger attributes but silently drops any leftover `property.*` sub-keys instead of forwarding them, so per-logger properties only work in the XML format. The documentation example is wrong as a result, and you're correct that we don't run tests against configuration snippets in the manual. **Workaround for now:** use the XML (or YAML/JSON) format if you need per-logger properties. On prioritisation: the `.properties` format is deprecated and will be replaced by a new Jackson-based properties `ConfigurationFactory` in 3.x, so we're not investing much in it. The fix here is small, forwarding the remaining properties to the `LoggerConfig` builder, analogous to what `createAppender` already does, and I'll add it to my list, but it won't be urgent. If anyone wants to send a PR in the meantime, the path through the code is clear from the issue description. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
