fix(inputs.upsd): Enforce string type for vendor and product IDs

[majiayu000]

Summary

The go.nut library auto-detects numeric-looking values using a regex and converts them to int64. This means a vendorid like "0764" becomes int64(764), but a non-numeric vendorid like "ABCD" stays a string. When both types of UPS devices report to the same InfluxDB bucket, you get type conflicts on the vendorid/productid fields.

This adds a stringFieldSet that forces ups.vendorid, ups.productid, driver.parameter.vendorid, and driver.parameter.productid to always be emitted as strings, following the same internal.ToString pattern already used for ups.firmware. The string check runs before the ForceFloat conversion so these fields aren't accidentally cast to float.

Also adds a test case with non-numeric vendor/product IDs to confirm type consistency.

Checklist

• No AI generated code was used in this PR

Related issues

resolves #18308

[srebhan]

Thanks for your contribution @majiayu000!

[skartikey]

Hi @majiayu000, thanks for this fix. After discussion with @srebhan, we'd like to land this behind a config option so users can migrate gracefully rather than getting a silent type change on upgrade. Here's the plan:

This PR (step 1): Add a config option (e.g. stringify_ids bool, defaulting to false to preserve current behavior). When the option is unset, keep the old numeric behavior but emit a warning in Init() noting that the default will flip in a future release and users should set the option explicitly. When set, apply the string-enforcement logic this PR already implements. Please also add a short note to the plugin README and a changelog entry flagging the upcoming default change.

Future PRs (for our tracking, not this one):

• Step 2: flip the default to string, emit an info log for users still not setting the option, mark the option deprecated for removal >= v1.x+5.
• Step 3: remove the option, keep only the enforce-string behavior.

One small thing worth knowing: because go.nut parses "0764" into int64(764) before we see it, the emitted string will be "764" rather than "0764". Leading zeros are lost and not recoverable at this layer. Probably worth a line in the README so users know.

Let me know if you have questions on the option naming or wording of the warning.

[majiayu000]

Thanks @skartikey, updated the PR to follow the 3-step migration plan. Pushed as 8f42f14.

Changes in this push:

• Added stringify_ids *bool config option, defaulting to unset. When unset, the legacy numeric behavior is preserved and a warning is emitted in Init() noting the default will flip in a future release. Setting it explicitly (either true or false) silences the warning.
• Gated the stringFieldSet enforcement behind stringify_ids = true. The check still runs before the ForceFloat branch so IDs aren't accidentally cast to float.
• Reverted CyberPowerSystems_CP900EPFCLCD_full/expected.out to the pre-PR integer form so it covers the default-off path.
• Added a CyberPowerSystems_CP900EPFCLCD_full_stringify_ids testcase with the same variables but stringify_ids = true in the conf, expecting strings.
• Documented the option in sample.conf and the README, including the "0764" → "764" leading-zero note you called out (good catch, thanks).

Happy to adjust the warning wording or the option name if you'd prefer something else. Also let me know if you'd rather have stringify_ids be a plain bool (default false, no warning) — I went with *bool so "unset" is distinguishable from an explicit false, which is what makes the "please set me explicitly" warning possible. If you'd rather keep the struct simpler I can switch to plain bool and drop the warning.

[telegraf-tiger]

Download PR build artifacts for linux_amd64.tar.gz, darwin_arm64.tar.gz, and windows_amd64.zip.
Downloads for additional architectures and packages are available below.

🥳 This pull request decreases the Telegraf binary size by -1.05 % for linux amd64 (new size: 300.7 MB, nightly size 303.9 MB)

📦 Click here to get additional PR build artifacts

Artifact URLs

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