From 1baf325149c5f51bcf0bd8e75c3ba409bad7c69a Mon Sep 17 00:00:00 2001
From: Lan Tian <xuyh0120@outlook.com>
Date: Sun, 17 Jan 2021 01:33:14 +0800
Subject: [PATCH] frontend: move redirect logic to HTML

---
 frontend/bindata/templates/page.tpl | 21 ++++++++++++++++++++-
 frontend/webserver.go               | 21 ---------------------
 2 files changed, 20 insertions(+), 22 deletions(-)

diff --git a/frontend/bindata/templates/page.tpl b/frontend/bindata/templates/page.tpl
index 017b51c..3fa4817 100644
--- a/frontend/bindata/templates/page.tpl
+++ b/frontend/bindata/templates/page.tpl
@@ -41,7 +41,7 @@
 		{{ if .IsWhois }}
 			{{ $target = .WhoisTarget }}
 		{{ end }}
-		<form class="form-inline" action="/redir" method="GET">
+		<form name="goto" class="form-inline" action="javascript:goto();">
 			<div class="input-group">
 				<select name="action" class="form-control">
 					{{ range $k, $v := .Options }}
@@ -64,5 +64,24 @@
 
 <script src="https://cdn.jsdelivr.net/npm/jquery@3.5.1/dist/jquery.min.js" integrity="sha256-9/aliU8dGd2tb6OSsuzixeV4y/faTqgFtohetphbbj0=" crossorigin="anonymous"></script>
 <script src="https://cdn.jsdelivr.net/npm/bootstrap@4.5.1/dist/js/bootstrap.min.js" integrity="sha256-0IiaoZCI++9oAAvmCb5Y0r93XkuhvJpRalZLffQXLok=" crossorigin="anonymous"></script>
+
+<script>
+function goto() {
+	let action = encodeURIComponent($('[name="action"]').val());
+	let server = encodeURIComponent($('[name="server"]').val());
+	let target = encodeURIComponent($('[name="target"]').val());
+	let url = "";
+
+	if (action == "whois") {
+		url = "/" + action + "/" + target;
+	} else if (action == "summary") {
+		url = "/" + action + "/" + server + "/";
+	} else {
+		url = "/" + action + "/" + server + "/" + target;
+	}
+
+	window.location.href = url;
+}
+</script>
 </body>
 </html>
diff --git a/frontend/webserver.go b/frontend/webserver.go
index 9dc36a9..11900c0 100644
--- a/frontend/webserver.go
+++ b/frontend/webserver.go
@@ -177,26 +177,6 @@ func webHandlerBGPMap(endpoint string, command string) func(w http.ResponseWrite
 	}
 }
 
-// redirect from the form input to a path style query
-func webHandlerNavbarFormRedirect(w http.ResponseWriter, r *http.Request) {
-	query := r.URL.Query()
-
-	action := query.Get("action")
-
-	switch action {
-	case "whois":
-		target := url.PathEscape(query.Get("target"))
-		http.Redirect(w, r, "/"+action+"/"+target, 302)
-	case "summary":
-		server := url.PathEscape(query.Get("server"))
-		http.Redirect(w, r, "/"+action+"/"+server+"/", 302)
-	default:
-		server := url.PathEscape(query.Get("server"))
-		target := url.PathEscape(query.Get("target"))
-		http.Redirect(w, r, "/"+action+"/"+server+"/"+target, 302)
-	}
-}
-
 // set up routing paths and start webserver
 func webServerStart() {
 
@@ -230,7 +210,6 @@ func webServerStart() {
 	http.HandleFunc("/generic/", webBackendCommunicator("bird", "generic"))
 	http.HandleFunc("/traceroute/", webBackendCommunicator("traceroute", "traceroute"))
 	http.HandleFunc("/whois/", webHandlerWhois)
-	http.HandleFunc("/redir", webHandlerNavbarFormRedirect)
 	http.HandleFunc("/telegram/", webHandlerTelegramBot)
 
 	// Start HTTP server