Skip to content

Commit

Permalink
CLI: Suppress printing out of password to console when using auth com…
Browse files Browse the repository at this point in the history
…mand (#2857)

---------

Co-authored-by: Eric Pugh <[email protected]>
(cherry picked from commit aadac83)
  • Loading branch information
malliaridis authored and epugh committed Dec 12, 2024
1 parent 532f091 commit a47942f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
2 changes: 2 additions & 0 deletions solr/CHANGES.txt
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,8 @@ Improvements
* SOLR-16470: Replication "fetch file" API now has a v2 equivalent, available at `GET /api/cores/coreName/replication/files/fileName`
(Matthew Biscocho via Jason Gerlowski)

* SOLR-17554: Suppress printing out of password to console when using auth CLI command. (Christos Malliaridis via Eric Pugh)


Optimizations
---------------------
Expand Down
5 changes: 1 addition & 4 deletions solr/core/src/java/org/apache/solr/cli/AuthTool.java
Original file line number Diff line number Diff line change
Expand Up @@ -539,10 +539,7 @@ private int handleBasicAuth(CommandLine cli) throws Exception {
includeFile.toPath(), basicAuthConfFile.getAbsolutePath(), null, cli);
final String successMessage =
String.format(
Locale.ROOT,
"Successfully enabled basic auth with username [%s] and password [%s].",
username,
password);
Locale.ROOT, "Successfully enabled basic auth with username [%s].", username);
echo(successMessage);
return 0;

Expand Down

0 comments on commit a47942f

Please sign in to comment.