neboskreb commented on PR #3638: URL: https://github.com/apache/logging-log4j2/pull/3638#issuecomment-2841819684
Okay, so checking the runtime platform appears ugly, because it requires copying the code from `SystemUtils` (which is available in `core` but not here in `api`) or referring to `core` as dependency - neither being a great idea... Therefore I decided, instead, to go with checking the capability of the instantiated `PrivateSecurityManager`. This looks more robust, because it's not the platform (we wouldn't be excited to add check `isOsXxxx` for every possible beast in the zoo) but the _capability_ we're interested in. I have completely rewritten the solution, and replaced the PR description. @ppkarwasz please check and say what you think? -- 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
