Merge pull request #15 from towalink/master

Increase consistency of path escaping;  support IPv6 addresses

frontend/bindata/templates/page.tpl

@@ -33,8 +33,8 @@
 			</li>
 			{{ range $k, $v := .Servers }}
 			<li class="nav-item">
-				<a class="nav-link{{ if eq $server $v }} active{{ end }}"
-					href="/{{ $option }}/{{ $v }}/{{ $target }}">{{ html $v }}</a>
+				<a class="nav-link{{ if eq $server $k }} active{{ end }}"
+					href="/{{ $option }}/{{ $k }}/{{ $target }}">{{ html $v }}</a>
 			</li>
 			{{ end }}
 		</ul>

frontend/lgproxy.go

@@ -5,6 +5,7 @@ import (
 	"net/http"
 	"net/url"
 	"strconv"
+	"strings"
 )
 
 type channelData struct {
@@ -36,6 +37,10 @@ func batchRequest(servers []string, endpoint string, command string) []string {
 		} else {
 			// Compose URL and send the request
 			hostname := server
+			hostname = url.PathEscape(hostname)
+			if strings.Contains(hostname, ":") {
+				hostname = "[" + hostname + "]"
+			}
 			if setting.domain != "" {
 				hostname += "." + setting.domain
 			}

frontend/render.go

@@ -5,6 +5,7 @@ import (
 	"errors"
 	"fmt"
 	"net/http"
+	"net/url"
 	"regexp"
 	"sort"
 	"strings"
@@ -47,23 +48,28 @@ func renderPageTemplate(w http.ResponseWriter, r *http.Request, title string, co
 	// Use a default URL if the request URL is too short
 	// The URL is for return to summary page
 	if len(split) < 2 {
-		path = "summary/" + strings.Join(setting.servers, "+") + "/"
+		path = "summary/" + url.PathEscape(strings.Join(setting.servers, "+")) + "/"
 	} else if len(split) == 2 {
 		path += "/"
 	}
 
 	split = strings.SplitN(path, "/", 3)
 
+	serversEscaped := map[string]string{}
+	for _, v := range setting.servers {
+		serversEscaped[url.PathEscape(v)] = v
+	}
+
 	args := TemplatePage{
 		Options:              optionsMap,
-		Servers:              setting.servers,
+		Servers:              serversEscaped,
 		AllServersLinkActive: strings.ToLower(split[1]) == strings.ToLower(strings.Join(setting.servers, "+")),
-		AllServersURL:        strings.Join(setting.servers, "+"),
+		AllServersURL:        url.PathEscape(strings.Join(setting.servers, "+")),
 		IsWhois:              isWhois,
 		WhoisTarget:          whoisTarget,
 
 		URLOption:  strings.ToLower(split[0]),
-		URLServer:  strings.ToLower(split[1]),
+		URLServer:  url.PathEscape(strings.ToLower(split[1])),
 		URLCommand: split[2],
 		Title:      setting.titleBrand + title,
 		Brand:      setting.navBarBrand,

frontend/template.go

@@ -11,7 +11,7 @@ import (
 type TemplatePage struct {
 	// Global options
 	Options map[string]string
-	Servers []string
+	Servers map[string]string
 
 	// Parameters related to current request
 	AllServersLinkActive bool

frontend/webserver.go

@@ -71,12 +71,7 @@ func webBackendCommunicator(endpoint string, command string) func(w http.Respons
 		split := strings.SplitN(r.URL.Path[1:], "/", 3)
 		var urlCommands string
 		if len(split) >= 3 {
-			tmp, err := url.PathUnescape(split[2])
-			if err != nil {
-				serverError(w, r)
-				return
-			}
-			urlCommands = tmp
+			urlCommands = split[2]
 		}
 
 		var backendCommand string
@@ -87,12 +82,7 @@ func webBackendCommunicator(endpoint string, command string) func(w http.Respons
 		}
 		backendCommand = strings.TrimSpace(backendCommand)
 
-		escapedServers, err := url.PathUnescape(split[1])
-		if err != nil {
-			serverError(w, r)
-			return
-		}
-		servers := strings.Split(escapedServers, "+")
+		servers := strings.Split(split[1], "+")
 
 		var responses []string = batchRequest(servers, endpoint, backendCommand)
 		var content string
@@ -182,7 +172,7 @@ func webServerStart() {
 
 	// redirect main page to all server summary
 	http.HandleFunc("/", func(w http.ResponseWriter, r *http.Request) {
-		http.Redirect(w, r, "/summary/"+strings.Join(setting.servers, "+"), 302)
+		http.Redirect(w, r, "/summary/"+url.PathEscape(strings.Join(setting.servers, "+")), 302)
 	})
 
 	// serve static pages using the AssetFS and bindata