dogstatsd: send length_prefix only for Unix Stream

nappairam · nappairam · commit 71af3c032b6c · 2025-03-04T10:54:15.000+08:00
Length prefix is only needed in case of Stream and not for Datagram whether it is Unix or UDP socket. Reference: https://github.com/DataDog/datadog-go/blob/bae3560e5c664d64b71c0e8ca89326afa362e12a/statsd/uds.go#L62
diff --git a/metrics-exporter-dogstatsd/src/forwarder/mod.rs b/metrics-exporter-dogstatsd/src/forwarder/mod.rs
@@ -118,7 +118,7 @@ impl ForwarderConfiguration {
             #[cfg(target_os = "linux")]
             RemoteAddr::Unix(_) => true,
             #[cfg(target_os = "linux")]
-            RemoteAddr::Unixgram(_) => true,
+            RemoteAddr::Unixgram(_) => false,
         }
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -118,7 +118,7 @@ impl ForwarderConfiguration {`
`118`	`118`	`#[cfg(target_os = "linux")]`
`119`	`119`	`RemoteAddr::Unix(_) => true,`
`120`	`120`	`#[cfg(target_os = "linux")]`
`121`		`- RemoteAddr::Unixgram(_) => true,`
	`121`	`+ RemoteAddr::Unixgram(_) => false,`
`122`	`122`	`}`
`123`	`123`	`}`
`124`	`124`	`}`