fixing insecure_skip_verify

[MyaLongmire]

Required for all PRs:

• Updated associated README.md.
• Wrote appropriate unit tests.

resolves #9389

removed tls from HTTP struct since it gets imported with HTTPClientConfig

[telegraf-tiger]

Looks like new artifacts were built from this PR. Get them here!

Artifact URLs

• darwin_amd64.tar.gz

• windows_amd64.zip

• windows_i386.zip

• aarch64.rpm

• linux_arm64.tar.gz

• arm64.deb

• linux_mipsel.tar.gz

• mipsel.deb

• armel.rpm

• linux_armel.tar.gz

• armel.deb

• ppc64le.rpm

• linux_ppc64le.tar.gz

• ppc64el.deb

• s390x.rpm

• linux_s390x.tar.gz

• s390x.deb

• x86_64.rpm

• freebsd_amd64.tar.gz

• linux_amd64.tar.gz

• amd64.deb

• i386.rpm

• freebsd_i386.tar.gz

• linux_i386.tar.gz

• i386.deb

• linux_mips.tar.gz

• mips.deb

• static_linux_amd64.tar.gz

• armv6hl.rpm

• freebsd_armv7.tar.gz

• linux_armhf.tar.gz

• armhf.deb

[sspaink]

We could add a unit test to verify the config to prevent this kind of regression in the future, something like this:

func TestConfig(t *testing.T) {
	c := config.NewConfig()
	require.NoError(t, c.LoadConfig("./testdata/http.toml"))
	output := outputs.Outputs["http"]().(*HTTP)
	output.URL = "http://127.0.0.1:8080/telegraf"
	c.Outputs[0].Output.(*HTTP).serializer = nil
	require.Equal(t, output, c.Outputs[0].Output)
}

edit: Don't think this is necessary for this change, maybe something to consider for the future

[popey]

Tested with the issue provided at this discussion and it works. Thanks @MyaLongmire !