implemented sorting headers, added ignoring (404) /favicon.ico requests

server.go

@@ -5,6 +5,7 @@ import (
 	"io/ioutil"
 	"net/http"
 	"os"
+	"sort"
 	"strings"
 	"time"
 )
@@ -15,13 +16,17 @@ var showEnvs = os.Getenv("YOSOY_SHOW_ENVS")
 var showFiles = os.Getenv("YOSOY_SHOW_FILES")
 
 func handler(w http.ResponseWriter, req *http.Request) {
+	if req.RequestURI == "/favicon.ico" {
+		w.WriteHeader(http.StatusNotFound)
+		return
+	}
+
 	remoteAddr := req.RemoteAddr
-	// LastIndex works better with IPv6
 	if index := strings.LastIndex(remoteAddr, ":"); index > 0 {
 		remoteAddr = remoteAddr[0:index]
 	}
 	fmt.Printf("[%v] - %v - %v - \"%v %v\"\n", hostname, time.Now().Format(time.RFC3339), remoteAddr, req.Method, req.RequestURI)
-	w.WriteHeader(200)
+	w.WriteHeader(http.StatusOK)
 	w.Header().Add("Content-Type", "text/plain")
 	fmt.Fprintf(w, "Request URI: %v\n", req.RequestURI)
 	fmt.Fprintf(w, "Hostname: %v\n", hostname)
@@ -30,9 +35,15 @@ func handler(w http.ResponseWriter, req *http.Request) {
 	fmt.Fprintf(w, "Called: %v\n", counter)
 	fmt.Fprintln(w)
 	fmt.Fprintf(w, "HTTP headers:\n")
-	for name, headers := range req.Header {
+	headers := make([]string, 0, len(req.Header))
+	for k := range req.Header {
+		headers = append(headers, k)
+	}
+	sort.Strings(headers)
+	for _, header := range headers {
+		headers := req.Header[header]
 		for _, h := range headers {
-			fmt.Fprintf(w, "%v: %v\n", name, h)
+			fmt.Fprintf(w, "%v: %v\n", header, h)
 		}
 	}
 	if strings.ToLower(showEnvs) == "true" || strings.ToLower(showEnvs) == "yes" || strings.ToLower(showEnvs) == "on" || showEnvs == "1" {