dsmiley commented on code in PR #3312: URL: https://github.com/apache/solr/pull/3312#discussion_r2286667041
########## solr/bin/solr.in.sh: ########## @@ -248,14 +248,14 @@ #SOLR_SECURITY_MANAGER_ENABLED=true # This variable provides you with the option to disable the Admin UI. If you uncomment the variable below and -# change the value to true. The option is configured as a system property as defined in SOLR_START_OPTS in the start +# change the value to false. The option is configured as a system property as defined in SOLR_START_OPTS in the start # scripts. -# SOLR_ADMIN_UI_DISABLED=false +# SOLR_UI_ENABLED=false Review Comment: I don't mean that. I mean the value in the comment should be inverted itself since you flipped the ENABLED|DISABLED. The commented value is supposed to be the default value. -- 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: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
