You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by bt...@apache.org on 2026/02/06 16:09:29 UTC
(hadoop) 01/01: Revert "YARN-11726: Add logging statements for successful and unsuccessful pa…"
This is an automated email from the ASF dual-hosted git repository.
bteke pushed a commit to branch revert-8224-exposing-password-retrieval-process
in repository https://gitbox.apache.org/repos/asf/hadoop.git
commit 653dd70bc389e2f5241fbf696be0ef12297be4f4
Author: Benjamin Teke <br...@users.noreply.github.com>
AuthorDate: Fri Feb 6 17:09:20 2026 +0100
Revert "YARN-11726: Add logging statements for successful and unsuccessful pa…"
This reverts commit c902faaef8b003843485350e330188a0ea32768a.
---
.../java/org/apache/hadoop/yarn/webapp/util/WebAppUtils.java | 4 ----
.../org/apache/hadoop/yarn/webapp/util/TestWebAppUtils.java | 10 ----------
2 files changed, 14 deletions(-)
diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/webapp/util/WebAppUtils.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/webapp/util/WebAppUtils.java
index e2fb9b31099..385ddffbb42 100644
--- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/webapp/util/WebAppUtils.java
+++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/webapp/util/WebAppUtils.java
@@ -533,14 +533,10 @@ static String getPassword(Configuration conf, String alias) {
char[] passchars = conf.getPassword(alias);
if (passchars != null) {
password = new String(passchars);
- LOG.debug("Successful password retrival for alias: {}", alias);
- } else {
- LOG.warn("Password retrieval failed, no password found for alias: {}", alias);
}
}
catch (IOException ioe) {
password = null;
- LOG.error("Unable to retrieve password for alias: {}", alias, ioe);
}
return password;
}
diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/test/java/org/apache/hadoop/yarn/webapp/util/TestWebAppUtils.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/test/java/org/apache/hadoop/yarn/webapp/util/TestWebAppUtils.java
index 56982623b0c..2d9c39aba8b 100644
--- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/test/java/org/apache/hadoop/yarn/webapp/util/TestWebAppUtils.java
+++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/test/java/org/apache/hadoop/yarn/webapp/util/TestWebAppUtils.java
@@ -109,16 +109,6 @@ void testGetPassword() throws Exception {
assertNull(WebAppUtils.getPassword(conf, "invalid-alias"));
}
- @Test
- void testGetPasswordIOException() throws Exception {
- Configuration mockConf = Mockito.mock(Configuration.class);
-
- Mockito.when(mockConf.getPassword("error-alias"))
- .thenThrow(new IOException("Simulated IO error"));
-
- assertNull(WebAppUtils.getPassword(mockConf, "error-alias"));
- }
-
@Test
void testLoadSslConfiguration() throws Exception {
Configuration conf = provisionCredentialsForSSL();
---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org