Explorar el Código

Don't SplitHostPort for meek_host_header

- There's no need to remove the port.

- Fixes: SplitHostHeader fails and causes
  "" to be logged when the original
  meek_host_header has no port, which it
  commonly does not when the port was 80.
Rod Hynes hace 7 años
padre
commit
feac4f46c5
Se han modificado 1 ficheros con 0 adiciones y 3 borrados
  1. 0 3
      psiphon/server/api.go

+ 0 - 3
psiphon/server/api.go

@@ -711,9 +711,6 @@ func getRequestLogFields(
 				} else {
 					logFields["meek_dial_domain"] = host
 				}
-			case "meek_host_header":
-				host, _, _ := net.SplitHostPort(strValue)
-				logFields[expectedParam.name] = host
 			case "upstream_proxy_type":
 				// Submitted value could be e.g., "SOCKS5" or "socks5"; log lowercase
 				logFields[expectedParam.name] = strings.ToLower(strValue)