Apply suggestions from code review

Co-authored-by: Saylor Berman <[email protected]>

Author: ciarams87

apis/v1alpha1/nginxproxy_types.go

@@ -36,7 +36,7 @@ type NginxProxySpec struct {
 	// Default is false, meaning http2 will be enabled for all servers.
 	//
 	// +optional
-	DisableHTTP2 bool `json:"disableHTTP2"`
+	DisableHTTP2 bool `json:"disableHTTP2,omitempty"`
 }
 
 // Telemetry specifies the OpenTelemetry configuration.

internal/mode/static/nginx/config/base_http_config_template.go

@@ -1,5 +1,5 @@
 package config
 
 const baseHTTPTemplateText = `
-{{- if .HTTP2 }}http2 on;{{- end }}
+{{- if .HTTP2 }}http2 on;{{ end }}
 `

internal/mode/static/state/dataplane/types.go

@@ -282,8 +282,8 @@ type SpanAttribute struct {
 	Value string
 }
 
-// BaseHTTPConfig holds the configuration options at the http context
+// BaseHTTPConfig holds the configuration options at the http context.
 type BaseHTTPConfig struct {
-	// HTTP2 specifies whether http2 should be enabled or disabled for all servers
+	// HTTP2 specifies whether http2 should be enabled or disabled for all servers.
 	HTTP2 bool
 }