You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by GitBox <gi...@apache.org> on 2022/12/01 16:34:42 UTC

[GitHub] [tomcat] isaacrivriv opened a new pull request, #572: 66370-Fix for Boolean System property lookup with Security Manager enabled

isaacrivriv opened a new pull request, #572:
URL: https://github.com/apache/tomcat/pull/572

   https://bz.apache.org/bugzilla/show_bug.cgi?id=66370


-- 
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: dev-unsubscribe@tomcat.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org

[GitHub] [tomcat] isaacrivriv commented on pull request #572: 66370-Fix for Boolean System property lookup with Security Manager enabled

Posted by GitBox <gi...@apache.org>.
isaacrivriv commented on PR #572:
URL: https://github.com/apache/tomcat/pull/572#issuecomment-1341392982

   Yes I saw that, my question was more as to why by default the property was made to disable the security manager. There are other areas of code where the security manager is used but there are no properties to disable them. This is why I suggest in this PR to update the property in order to keep the same behavior as the other areas of code that use the security manager if enabled by default. We could use `org.apache.el.GET_CLASSLOADER_IGNORE_PRIVILEGED` instead of `org.apache.el.GET_CLASSLOADER_USE_PRIVILEGED` to disable or enable the security manager in this case which would still have the benefits listed in that issue to not use the security manager but would need to be manually set.


-- 
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: dev-unsubscribe@tomcat.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org

[GitHub] [tomcat] markt-asf closed pull request #572: 66370-Fix for Boolean System property lookup with Security Manager enabled

Posted by GitBox <gi...@apache.org>.
markt-asf closed pull request #572: 66370-Fix for Boolean System property lookup with Security Manager enabled
URL: https://github.com/apache/tomcat/pull/572


-- 
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: dev-unsubscribe@tomcat.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org

[GitHub] [tomcat] markt-asf commented on pull request #572: 66370-Fix for Boolean System property lookup with Security Manager enabled

Posted by GitBox <gi...@apache.org>.
markt-asf commented on PR #572:
URL: https://github.com/apache/tomcat/pull/572#issuecomment-1371172938

   The PR will not be applied.


-- 
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: dev-unsubscribe@tomcat.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org

[GitHub] [tomcat] lalo-mx commented on pull request #572: 66370-Fix for Boolean System property lookup with Security Manager enabled

Posted by GitBox <gi...@apache.org>.
lalo-mx commented on PR #572:
URL: https://github.com/apache/tomcat/pull/572#issuecomment-1341125405

   See https://bz.apache.org/bugzilla/show_bug.cgi?id=66294


-- 
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: dev-unsubscribe@tomcat.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org

[GitHub] [tomcat] ChristopherSchultz commented on pull request #572: 66370-Fix for Boolean System property lookup with Security Manager enabled

Posted by GitBox <gi...@apache.org>.
ChristopherSchultz commented on PR #572:
URL: https://github.com/apache/tomcat/pull/572#issuecomment-1341716603

   You should read both https://lists.apache.org/thread/66djc4j4ybz45ponly0x5g94oyt844cm and https://lists.apache.org/thread/7w4x90zwp3qhn9qopbhmg5wponcc468n in their entirety to see the thought-process(es) here.


-- 
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: dev-unsubscribe@tomcat.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org