vy commented on PR #3532: URL: https://github.com/apache/logging-log4j2/pull/3532#issuecomment-2742939724
@kazariad, I internally discussed this issue with @ppkarwasz, and we decided to not proceed with this PR due to following reasons: 1. Putting aside your changes, UUID Pattern Converter is not in its ideal shape. The way it uses properties to pass temporal state, branches at runtime on `UuidPatternConverter::isRandom`, etc. needs to be improved. Further, with some minor effort, it can even be made garbage-free. 2. I'm against copy-pasting a full-blown hashing algorithm – this is error prone and can pose security risks if used unwisely. In short, it is a liability that I prefer to avoid. 3. There is more to this issue than it appears and, as the only active maintainers, @ppkarwasz and I would rather be busy with other more pressing issues. In [our earlier discussion](/apache/logging-log4j2/discussions/3503), @ppkarwasz and I pointed you several potential paths. If you decide to go that route, and publish the code yourself (or your organization), then we would be happy to refer to your project and its Maven artifacts in our official documentation. Another route I can personally suggest you is to seek for [commercial support offered by Log4j maintainers](https://logging.apache.org/support.html#commercial). -- 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
