From fdb38227564c1af40cbfb97df420b23eb04c002b Mon Sep 17 00:00:00 2001
From: Rohit Agarwal <rohita@qubole.com>
Date: Wed, 16 Dec 2015 19:04:33 -0800
Subject: [PATCH] [SPARK-12186][WEB UI] Send the complete request URI including
the query string when redirecting.
Author: Rohit Agarwal <rohita@qubole.com>
Closes #10180 from mindprince/SPARK-12186.
---
.../scala/org/apache/spark/deploy/history/HistoryServer.scala | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/core/src/main/scala/org/apache/spark/deploy/history/HistoryServer.scala b/core/src/main/scala/org/apache/spark/deploy/history/HistoryServer.scala
index d4f327cc58..f31fef0ecc 100644
--- a/core/src/main/scala/org/apache/spark/deploy/history/HistoryServer.scala
+++ b/core/src/main/scala/org/apache/spark/deploy/history/HistoryServer.scala
@@ -103,7 +103,9 @@ class HistoryServer(
// Note we don't use the UI retrieved from the cache; the cache loader above will register
// the app's UI, and all we need to do is redirect the user to the same URI that was
// requested, and the proper data should be served at that point.
- res.sendRedirect(res.encodeRedirectURL(req.getRequestURI()))
+ // Also, make sure that the redirect url contains the query string present in the request.
+ val requestURI = req.getRequestURI + Option(req.getQueryString).map("?" + _).getOrElse("")
+ res.sendRedirect(res.encodeRedirectURL(requestURI))
}
// SPARK-5983 ensure TRACE is not supported
--
GitLab