diff options
author | Artur Sukhenko <artur.sukhenko@gmail.com> | 2016-08-06 04:41:47 +0100 |
---|---|---|
committer | Sean Owen <sowen@cloudera.com> | 2016-08-06 04:41:47 +0100 |
commit | 14dba45208d8a5511be2cf8ddf22e688ef141e88 (patch) | |
tree | 690b0e6d5d9c701659e09158b9a2e120a7d16e70 | |
parent | 55d6dad6f21dd4d50d168f6392242aa8e24b774a (diff) | |
download | spark-14dba45208d8a5511be2cf8ddf22e688ef141e88.tar.gz spark-14dba45208d8a5511be2cf8ddf22e688ef141e88.tar.bz2 spark-14dba45208d8a5511be2cf8ddf22e688ef141e88.zip |
[SPARK-16796][WEB UI] Mask spark.authenticate.secret on Spark environ…
## What changes were proposed in this pull request?
Mask `spark.authenticate.secret` on Spark environment page (Web UI).
This is addition to https://github.com/apache/spark/pull/14409
## How was this patch tested?
`./dev/run-tests`
[info] ScalaTest
[info] Run completed in 1 hour, 8 minutes, 38 seconds.
[info] Total number of tests run: 2166
[info] Suites: completed 65, aborted 0
[info] Tests: succeeded 2166, failed 0, canceled 0, ignored 590, pending 0
[info] All tests passed.
Author: Artur Sukhenko <artur.sukhenko@gmail.com>
Closes #14484 from Devian-ua/SPARK-16796.
-rw-r--r-- | core/src/main/scala/org/apache/spark/ui/env/EnvironmentPage.scala | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/core/src/main/scala/org/apache/spark/ui/env/EnvironmentPage.scala b/core/src/main/scala/org/apache/spark/ui/env/EnvironmentPage.scala index 22136a6f10..9f6e9a6c90 100644 --- a/core/src/main/scala/org/apache/spark/ui/env/EnvironmentPage.scala +++ b/core/src/main/scala/org/apache/spark/ui/env/EnvironmentPage.scala @@ -27,7 +27,9 @@ private[ui] class EnvironmentPage(parent: EnvironmentTab) extends WebUIPage("") private val listener = parent.listener private def removePass(kv: (String, String)): (String, String) = { - if (kv._1.toLowerCase.contains("password")) (kv._1, "******") else kv + if (kv._1.toLowerCase.contains("password") || kv._1.toLowerCase.contains("secret")) { + (kv._1, "******") + } else kv } def render(request: HttpServletRequest): Seq[Node] = { |