Skip to content
Snippets Groups Projects
Commit 76e1f165 authored by sarutak's avatar sarutak Committed by Marcelo Vanzin
Browse files

[SPARK-18762][WEBUI] Web UI should be http:4040 instead of https:4040

## What changes were proposed in this pull request?

When SSL is enabled, the Spark shell shows:
```
Spark context Web UI available at https://192.168.99.1:4040


```
This is wrong because 4040 is http, not https. It redirects to the https port.
More importantly, this introduces several broken links in the UI. For example, in the master UI, the worker link is https:8081 instead of http:8081 or https:8481.

CC: mengxr liancheng

I manually tested accessing by accessing MasterPage, WorkerPage and HistoryServer with SSL enabled.

Author: sarutak <sarutak@oss.nttdata.co.jp>

Closes #16190 from sarutak/SPARK-18761.

(cherry picked from commit bb94f61a)
Signed-off-by: default avatarMarcelo Vanzin <vanzin@cloudera.com>
parent acb6ac5d
No related branches found
No related tags found
No related merge requests found
......@@ -187,8 +187,7 @@ private[deploy] class Worker(
webUi = new WorkerWebUI(this, workDir, webUiPort)
webUi.bind()
val scheme = if (webUi.sslOptions.enabled) "https" else "http"
workerWebUiUrl = s"$scheme://$publicAddress:${webUi.boundPort}"
workerWebUiUrl = s"http://$publicAddress:${webUi.boundPort}"
registerWithMaster()
metricsSystem.registerSource(workerSource)
......
......@@ -147,10 +147,7 @@ private[spark] abstract class WebUI(
}
/** Return the url of web interface. Only valid after bind(). */
def webUrl: String = {
val protocol = if (sslOptions.enabled) "https" else "http"
s"$protocol://$publicHostName:$boundPort"
}
def webUrl: String = s"http://$publicHostName:$boundPort"
/** Return the actual port to which this server is bound. Only valid after bind(). */
def boundPort: Int = serverInfo.map(_.boundPort).getOrElse(-1)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment