Skip to content

Commit 6d6e5fb

Browse files
upgrade goflow2 to fix netflow parser issue (#49888)
### What does this PR do? This PR integrates the change from this PR netsampler/goflow2#503 to fix an issue we are seeing in some customer environments when parsing incoming packets. ### Motivation Addresses an issue reported by a customer ### Describe how you validated your changes * Replayed packets from a pcap to validate that they are now parsed as expected ### Additional Notes Co-authored-by: garrison.stauffer <garrison.stauffer@datadoghq.com>
1 parent e083f5b commit 6d6e5fb

3 files changed

Lines changed: 19 additions & 6 deletions

File tree

go.mod

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -283,7 +283,7 @@ require (
283283
github.com/miekg/dns v1.1.72
284284
github.com/moby/sys/mountinfo v0.7.2
285285
github.com/mohae/deepcopy v0.0.0-20170929034955-c48cc78d4826
286-
github.com/netsampler/goflow2 v1.3.3
286+
github.com/netsampler/goflow2 v1.3.8-0.20260412031118-ad727784ae6f
287287
github.com/oliveagle/jsonpath v0.0.0-20180606110733-2e52cf6e6852
288288
github.com/open-policy-agent/opa v1.7.1
289289
github.com/open-telemetry/opentelemetry-collector-contrib/pkg/datadog v0.150.0
@@ -654,7 +654,7 @@ require (
654654
github.com/kylelemons/godebug v1.1.0 // indirect
655655
github.com/leodido/go-syslog/v4 v4.3.0 // indirect
656656
github.com/leodido/ragel-machinery v0.0.0-20190525184631-5f46317e436b // indirect
657-
github.com/libp2p/go-reuseport v0.2.0 // indirect
657+
github.com/libp2p/go-reuseport v0.3.0 // indirect
658658
github.com/lightstep/go-expohisto v1.0.0 // indirect
659659
github.com/linode/go-metadata v0.2.4 // indirect
660660
github.com/linode/linodego v1.66.0 // indirect

go.sum

Lines changed: 4 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
# Each section from every release note are combined when the
2+
# CHANGELOG.rst is rendered. So the text needs to be worded so that
3+
# it does not depend on any information only available in another
4+
# section. This may mean repeating some details, but each section
5+
# must be readable independently of the other.
6+
#
7+
# Each section note must be formatted as reStructuredText.
8+
---
9+
fixes:
10+
- |
11+
Fixes an issue with the NetFlow collector where certain packets would be dropped, producing
12+
error logs and lost data. The issue was caused when a packet had trailing padding
13+
which was not properly handled.

0 commit comments

Comments
 (0)