ppkarwasz commented on PR #3638:
URL: https://github.com/apache/logging-log4j2/pull/3638#issuecomment-2841620874

   > But I'm not quite sure I understand what is the blocker here, sorry. What 
can I do to unblock the progress?
   
   Currently you are proposing a new 
`PrivateSecurityManagerStackTraceUtil#disable()` method that can be used by 
other libraries to disable the security manager.
   
   My point is that `PrivateSecurityManagerStackTraceUtil` will **never** work 
on Android, so we might as well always disable the class on that platform. More 
concretely the static initialization code:
   
   
https://github.com/apache/logging-log4j2/blob/7acbc486854cd9b62184883ef9ee5973e1ef1b8b/log4j-api/src/main/java/org/apache/logging/log4j/util/PrivateSecurityManagerStackTraceUtil.java#L31-L44
   
   should:
   
   - check if the Java property `java.vendor` contains the word `Android`.
   - if we are running on Android set `SECURITY_MANAGER` to `null`.
   - otherwise proceed as before.


-- 
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