MarcusSorealheis commented on a change in pull request #1141: SOLR-14147 change 
the Security manager to default to true.
URL: https://github.com/apache/lucene-solr/pull/1141#discussion_r363549534
 
 

 ##########
 File path: solr/bin/solr
 ##########
 @@ -2084,14 +2084,14 @@ else
   REMOTE_JMX_OPTS=()
 fi
 
-# Enable java security manager (limiting filesystem access and other things)
-if [ "$SOLR_SECURITY_MANAGER_ENABLED" == "true" ]; then
+# Disable java security manager (allowing filesystem access and other things)
+if [ "$SOLR_SECURITY_MANAGER_ENABLED" == "false" ]; then
 
 Review comment:
   should be resolved?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to