Merge pull request #1740 from arkodg/clarify-port-redirect-default

Clarify semantics when port_redirect is unset

apis/v1beta1/httproute_types.go

@@ -882,7 +882,8 @@ type HTTPRequestRedirectFilter struct {
 
 	// Hostname is the hostname to be used in the value of the `Location`
 	// header in the response.
-	// When empty, the hostname of the request is used.
+	// When empty, the hostname in the `Host` header
+	// of the request is used.
 	//
 	// Support: Core
 	//
@@ -901,7 +902,10 @@ type HTTPRequestRedirectFilter struct {
 
 	// Port is the port to be used in the value of the `Location`
 	// header in the response.
-	// When empty, port (if specified) of the request is used.
+	// When empty, the Gateway Listener port is used.
+	// In all cases, when the protocol is http and the port is port 80,
+	// then the port must be omitted. Similarly, when the protocol is
+	// https and the port is port 443, then the port must also be omitted.
 	//
 	// Support: Extended
 	//