epugh commented on code in PR #3223:
URL: https://github.com/apache/solr/pull/3223#discussion_r1987194246


##########
solr/core/src/java/org/apache/solr/cli/AuthTool.java:
##########
@@ -96,7 +98,7 @@ public class AuthTool extends ToolBase {
           .argName("FILE")
           .required()
           .desc(
-              "This is where any authentication related configuration files, 
if any, would be placed.")
+              "This is where any authentication related configuration files, 
if any, would be placed.  Defaults to $SOLR_HOME.")

Review Comment:
   I don't think using these outside the CLI scripts is actually a true 
scenario...   In fact, I think if folks are using these tools outside of the 
bin/solr scripts (like some folks have their own start scripts) it points to 
weaknesses in bin/solr that we need to improve! 



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to