fix(inputs.bind): Convert counters to uint64

Author: srebhan

CHANGELOG.md

@@ -1,6 +1,17 @@
 <!-- markdownlint-disable MD024 -->
 # Changelog
 
+## Unreleased
+
+### Important Changes
+
+- PR [#](https://github.com/influxdata/telegraf/pull/) changes the internal
+  counters of the Bind plugin to unsigned integers matching the server
+  implementation. We keep backward compatibility by setting
+  `report_counters_as_int` to `true` by default to avoid type conflicts on the
+  output side. However, you should change this setting to `false` as soon as
+  possible to avoid invalid values and parsing errors with the v3 XML statistics.
+
 ## v1.32.1 [2024-10-07]
 
 ### Important Changes

plugins/inputs/bind/README.md

@@ -35,6 +35,11 @@ See the [CONFIGURATION.md][CONFIGURATION.md] for more details.
   # gather_memory_contexts = false
   # gather_views = false
 
+  ## Report xml v3 counters as integers instead of unsigned for backward
+  ## compatibility. Set this to false as soon as possible!
+  ## Values are clipped if exceeding the integer range.
+  # report_counters_as_int = true
+
   ## Timeout for http requests made by bind nameserver
   # timeout = "4s"
 ```

plugins/inputs/bind/bind.go

@@ -18,10 +18,11 @@ import (
 var sampleConfig string
 
 type Bind struct {
-	Urls                 []string
-	GatherMemoryContexts bool
-	GatherViews          bool
+	Urls                 []string        `tom:"urls"`
+	GatherMemoryContexts bool            `tom:"gather_memory_contexts"`
+	GatherViews          bool            `tom:"gather_views"`
 	Timeout              config.Duration `toml:"timeout"`
+	CountersAsInt        bool            `toml:"report_counters_as_int"`
 
 	client http.Client
 }
@@ -84,5 +85,5 @@ func (b *Bind) gatherURL(addr *url.URL, acc telegraf.Accumulator) error {
 }
 
 func init() {
-	inputs.Add("bind", func() telegraf.Input { return &Bind{} })
+	inputs.Add("bind", func() telegraf.Input { return &Bind{CountersAsInt: true} })
 }