This is an automated email from the ASF dual-hosted git repository.
epugh pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/solr.git
The following commit(s) were added to refs/heads/main by this push:
new aadac837034 CLI: Suppress printing out of password to console when
using auth command (#2857)
aadac837034 is described below
commit aadac837034f7996876d91526afabf01d3bfcf37
Author: Christos Malliaridis <malliari...@apache.org>
AuthorDate: Thu Dec 12 17:21:26 2024 +0200
CLI: Suppress printing out of password to console when using auth command
(#2857)
---------
Co-authored-by: Eric Pugh <ep...@opensourceconnections.com>
---
solr/CHANGES.txt | 2 ++
solr/core/src/java/org/apache/solr/cli/AuthTool.java | 5 +----
2 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/solr/CHANGES.txt b/solr/CHANGES.txt
index f8a7ae0de27..e46e9d9d1ae 100644
--- a/solr/CHANGES.txt
+++ b/solr/CHANGES.txt
@@ -215,6 +215,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
---------------------
diff --git a/solr/core/src/java/org/apache/solr/cli/AuthTool.java
b/solr/core/src/java/org/apache/solr/cli/AuthTool.java
index e4f9359d93b..6a71164a03d 100644
--- a/solr/core/src/java/org/apache/solr/cli/AuthTool.java
+++ b/solr/core/src/java/org/apache/solr/cli/AuthTool.java
@@ -282,10 +282,7 @@ public class AuthTool extends ToolBase {
updateIncludeFileEnableAuth(includeFile, basicAuthConfFile);
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;
case "disable":
