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_r363077445
########## File path: solr/bin/solr.in.sh ########## @@ -229,4 +229,4 @@ # Runtime properties are passed to the security policy file (server/etc/security.policy) # You can also tweak via standard JDK files such as ~/.java.policy, see https://s.apache.org/java8policy # This is experimental! It may not work at all with Hadoop/HDFS features. -#SOLR_SECURITY_MANAGER_ENABLED=false +SOLR_SECURITY_MANAGER_ENABLED=true Review comment: Thanks @rmuir for the information. I can roll back one commit to get back to there. I started with it, after reading the file and then uncommented this morning. I think that maybe we can try to make this functionality more apparent to readers through a simple clause, which I will add to the PR for the top of file, in the same paragraph. ---------------------------------------------------------------- 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
