HTTP Server keepalive.max_connection_age_secs can't be set to null

[andrewma2]

A note for the community

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment

Problem

According to the update guide the newly added keepalive feature for HTTP server sources can be disabled by setting keepalive.max_connection_age_secs to null.

However, vector hits a parse error when we set this value in our JSON config (says it encountered an invalid TOML value despite us passing a JSON config)

Configuration

Basic test config (file called `vector.json`)


{
	"sources": {
		"http": {
			"address": "0.0.0.0:443",
			"type": "http_server",
			"keepalive": {
				"max_connection_age_secs": null
			}
		}
	},
	"sinks": {
		"blackhole": {
			"inputs": ["http"],
			"type": "blackhole"
		}
	}
}

Version

0.35.0

Debug Output

Running
`vector --config vector.json`

produces this output 

2024-01-17T19:46:11.429210Z  INFO vector::app: Log level is enabled. level="info"
2024-01-17T19:46:11.434724Z  INFO vector::app: Loading configs. paths=["vector.json"]
2024-01-17T19:46:11.442812Z ERROR vector::cli: Configuration error. error=invalid type: null, expected any valid TOML value at line 7 column 35

Example Data

No response

Additional Context

Initial support thread on Discord

References

No response