Merge pull request #655 from yeya24/change-omit-empty

Arthur Silva Sens · web-flow · commit 04635d2962f9 · 2024-06-15T17:45:47.000-03:00
Set http_headers to be omit empty
diff --git a/config/http_config.go b/config/http_config.go
@@ -323,7 +323,7 @@ type HTTPClientConfig struct {
 	ProxyConfig `yaml:",inline"`
 	// HTTPHeaders specify headers to inject in the requests. Those headers
 	// could be marshalled back to the users.
-	HTTPHeaders *Headers `yaml:"http_headers" json:"http_headers"`
+	HTTPHeaders *Headers `yaml:"http_headers,omitempty" json:"http_headers,omitempty"`
 }
 
 // SetDirectory joins any relative file paths with dir.
diff --git a/config/http_config_test.go b/config/http_config_test.go
@@ -2023,7 +2023,6 @@ func TestHTTPClientConfig_Marshal(t *testing.T) {
 proxy_url: "http://localhost:8080"
 follow_redirects: false
 enable_http2: false
-http_headers: null
 `, string(actualYAML))
 
 			// Unmarshalling the YAML should get the same struct in input.
@@ -2040,8 +2039,7 @@ http_headers: null
 				"proxy_url":"http://localhost:8080",
 				"tls_config":{"insecure_skip_verify":false},
 				"follow_redirects":false,
-				"enable_http2":false,
-				"http_headers":null
+				"enable_http2":false
 			}`, string(actualJSON))
 
 			// Unmarshalling the JSON should get the same struct in input.

Original file line number	Diff line number	Diff line change
`@@ -323,7 +323,7 @@ type HTTPClientConfig struct {`
`323`	`323`	ProxyConfig `yaml:",inline"`
`324`	`324`	`// HTTPHeaders specify headers to inject in the requests. Those headers`
`325`	`325`	`// could be marshalled back to the users.`
`326`		- HTTPHeaders *Headers `yaml:"http_headers" json:"http_headers"`
	`326`	+ HTTPHeaders *Headers `yaml:"http_headers,omitempty" json:"http_headers,omitempty"`
`327`	`327`	`}`
`328`	`328`
`329`	`329`	`// SetDirectory joins any relative file paths with dir.`