From 716fb979917838d11c4090e01d09a3bc51dde83d Mon Sep 17 00:00:00 2001 From: Leo Ariel Arias Date: Tue, 2 Apr 2024 13:37:37 +1300 Subject: [PATCH 1/3] fix: enabling gzip was causing eof errors upon decompression --- .../awskinesisexporter/internal/compress/compresser.go | 7 +++++++ .../internal/compress/noop_compression.go | 2 ++ 2 files changed, 9 insertions(+) diff --git a/exporter/awskinesisexporter/internal/compress/compresser.go b/exporter/awskinesisexporter/internal/compress/compresser.go index f3060253678f..9197d19e3556 100644 --- a/exporter/awskinesisexporter/internal/compress/compresser.go +++ b/exporter/awskinesisexporter/internal/compress/compresser.go @@ -16,6 +16,7 @@ type bufferedResetWriter interface { Write(p []byte) (int, error) Flush() error Reset(newWriter io.Writer) + Close() error } type Compressor interface { @@ -74,5 +75,11 @@ func (c *compressor) Do(in []byte) ([]byte, error) { return nil, err } + if closer, ok := c.compression.(io.Closer); ok { + if err := closer.Close(); err != nil { + return nil, err + } + } + return buf.Bytes(), nil } diff --git a/exporter/awskinesisexporter/internal/compress/noop_compression.go b/exporter/awskinesisexporter/internal/compress/noop_compression.go index 0c8abf17b329..83175a4e47a2 100644 --- a/exporter/awskinesisexporter/internal/compress/noop_compression.go +++ b/exporter/awskinesisexporter/internal/compress/noop_compression.go @@ -26,3 +26,5 @@ func (n noop) Write(p []byte) (int, error) { func (n noop) Flush() error { return nil } + +func (n *noop) Close() error { return n.Close() } From 0f3452c9d127a1efb1431e818d282ae7ce4cbb79 Mon Sep 17 00:00:00 2001 From: Leo Ariel Arias Date: Tue, 2 Apr 2024 14:56:59 +1300 Subject: [PATCH 2/3] added changelog for fix --- .chloggen/fix-compression-eof.yaml | 27 +++++++++++++++++++++++++++ 1 file changed, 27 insertions(+) create mode 100644 .chloggen/fix-compression-eof.yaml diff --git a/.chloggen/fix-compression-eof.yaml b/.chloggen/fix-compression-eof.yaml new file mode 100644 index 000000000000..2a3a34d537aa --- /dev/null +++ b/.chloggen/fix-compression-eof.yaml @@ -0,0 +1,27 @@ +# Use this changelog template to create an entry for release notes. + +# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix' +change_type: bug_fix + +# The name of the component, or a single word describing the area of concern, (e.g. filelogreceiver) +component: exporter/awskinesisexporter + +# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`). +note: "Fixed issue with compression what was causing EOF exceptions when attempting to decompress the payload" + +# Mandatory: One or more tracking issues related to the change. You can use the PR number here if no issue exists. +issues: [32081] + +# (Optional) One or more lines of additional information to render under the primary note. +# These lines will be padded with 2 spaces and then inserted directly into the document. +# Use pipe (|) for multiline entries. +subtext: + +# If your change doesn't affect end users or the exported elements of any package, +# you should instead start your pull request title with [chore] or use the "Skip Changelog" label. +# Optional: The change log or logs in which this entry should be included. +# e.g. '[user]' or '[user, api]' +# Include 'user' if the change is relevant to end users. +# Include 'api' if there is a change to a library API. +# Default: '[user]' +change_logs: [user] From f1e16d6b7598c6b14c8976dcfee571475009f408 Mon Sep 17 00:00:00 2001 From: Leo Ariel Arias Date: Tue, 2 Apr 2024 16:39:58 +1300 Subject: [PATCH 3/3] fixed wrong noop implementation for compressor Close --- .../awskinesisexporter/internal/compress/noop_compression.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/exporter/awskinesisexporter/internal/compress/noop_compression.go b/exporter/awskinesisexporter/internal/compress/noop_compression.go index 83175a4e47a2..be594b6214f1 100644 --- a/exporter/awskinesisexporter/internal/compress/noop_compression.go +++ b/exporter/awskinesisexporter/internal/compress/noop_compression.go @@ -27,4 +27,4 @@ func (n noop) Flush() error { return nil } -func (n *noop) Close() error { return n.Close() } +func (n *noop) Close() error { return nil }