fix: Linter fixes for plugins/inputs/[h-j]*

[zak-pawel]

Following findings were fixed:

plugins/inputs/haproxy/haproxy_test.go:137:3                             revive       defer: prefer not to defer inside loops
plugins/inputs/hddtemp/hddtemp_test.go:36:2                              revive       import-shadowing: The name 'hddtemp' shadows an import name
plugins/inputs/http/http_test.go:118:2                                   revive       import-shadowing: The name 'url' shadows an import name
plugins/inputs/http/http_test.go:119:2                                   revive       import-shadowing: The name 'plugin' shadows an import name
plugins/inputs/http/http_test.go:146:2                                   revive       import-shadowing: The name 'plugin' shadows an import name
plugins/inputs/http/http_test.go:172:2                                   revive       import-shadowing: The name 'url' shadows an import name
plugins/inputs/http/http_test.go:30:2                                    revive       import-shadowing: The name 'url' shadows an import name
plugins/inputs/http/http_test.go:31:2                                    revive       import-shadowing: The name 'plugin' shadows an import name
plugins/inputs/http/http_test.go:72:2                                    revive       import-shadowing: The name 'url' shadows an import name
plugins/inputs/http/http_test.go:73:2                                    revive       import-shadowing: The name 'plugin' shadows an import name
plugins/inputs/http/http_test.go:95:2                                    revive       import-shadowing: The name 'url' shadows an import name
plugins/inputs/http/http_test.go:96:2                                    revive       import-shadowing: The name 'plugin' shadows an import name
plugins/inputs/http_listener_v2/http_listener_v2_test.go:372:24          bodyclose    response body must be closed
plugins/inputs/icinga2/icinga2.go:155:2                                  revive       import-shadowing: The name 'url' shadows an import name
plugins/inputs/icinga2/icinga2.go:85:3                                   revive       import-shadowing: The name 'url' shadows an import name
plugins/inputs/influxdb_listener/influxdb_listener_test.go:417:24        bodyclose    response body must be closed
plugins/inputs/influxdb_listener/influxdb_listener_test.go:526:24        bodyclose    response body must be closed
plugins/inputs/influxdb_v2_listener/influxdb_v2_listener_test.go:375:24  bodyclose    response body must be closed
plugins/inputs/interrupts/interrupts.go:118:3                            revive       defer: prefer not to defer inside loops
plugins/inputs/ipmi_sensor/ipmi.go:7:2                                   revive       imports-blacklist: should not use the following blacklisted import: "log"
plugins/inputs/ipmi_sensor/ipmi_test.go:374:2                            staticcheck  SA4006: this value of `args` is never used
plugins/inputs/ipmi_sensor/ipmi_test.go:383:3                            revive       deep-exit: calls to os.Exit only in main() or init() functions
plugins/inputs/ipmi_sensor/ipmi_test.go:385:2                            revive       deep-exit: calls to os.Exit only in main() or init() functions
plugins/inputs/ipmi_sensor/ipmi_test.go:571:2                            staticcheck  SA4006: this value of `args` is never used
plugins/inputs/ipmi_sensor/ipmi_test.go:580:3                            revive       deep-exit: calls to os.Exit only in main() or init() functions
plugins/inputs/ipmi_sensor/ipmi_test.go:582:2                            revive       deep-exit: calls to os.Exit only in main() or init() functions
plugins/inputs/jenkins/jenkins_test.go:47:3                              revive       unexported-naming: the symbol URL is local, its name should start with a lowercase letter
plugins/inputs/jolokia2/client.go:98:16                                  revive       import-shadowing: The name 'url' shadows an import name
plugins/inputs/jolokia2/gatherer.go:83:4                                 revive       unnecessary-stmt: omit unnecessary break at the end of case clause
plugins/inputs/jolokia2/jolokia_agent.go:100:9                           revive       import-shadowing: The name 'config' shadows an import name
plugins/inputs/jolokia2/jolokia_proxy.go:96:9                            revive       import-shadowing: The name 'config' shadows an import name
plugins/inputs/jolokia2/jolokia_test.go:752:2                            predeclared  variable complex has same name as predeclared identifier
plugins/inputs/jti_openconfig_telemetry/openconfig_telemetry.go:45:17    gosimple     S1007: should use raw string (`...`) with regexp.MustCompile to avoid having to escape twice

[srebhan]

Looks good. I only have two minor comments... :-)

[powersj]

Thanks for this! If you fix up the two items from Sven, I'll be +1 on this!

[telegraf-tiger]

☺️ This pull request doesn't change the Telegraf binary size

📦 Looks like new artifacts were built from this PR.

Expand this list to get them here! 🐯

Artifact URLs

DEB	RPM	TAR GZ	ZIP
amd64.deb	aarch64.rpm	darwin_amd64.tar.gz	windows_amd64.zip
arm64.deb	armel.rpm	freebsd_amd64.tar.gz	windows_i386.zip
armel.deb	armv6hl.rpm	freebsd_armv7.tar.gz
armhf.deb	i386.rpm	freebsd_i386.tar.gz
i386.deb	ppc64le.rpm	linux_amd64.tar.gz
mips.deb	s390x.rpm	linux_arm64.tar.gz
mipsel.deb	x86_64.rpm	linux_armel.tar.gz
ppc64el.deb		linux_armhf.tar.gz
s390x.deb		linux_i386.tar.gz
		linux_mips.tar.gz
		linux_mipsel.tar.gz
		linux_ppc64le.tar.gz
		linux_s390x.tar.gz
		static_linux_amd64.tar.gz