Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[exporter/splunkhec] Do not allow nil event values when exporting #29551

Closed
atoulme opened this issue Nov 29, 2023 · 2 comments · Fixed by #29552
Closed

[exporter/splunkhec] Do not allow nil event values when exporting #29551

atoulme opened this issue Nov 29, 2023 · 2 comments · Fixed by #29552
Assignees
@atoulme
Labels
bug Something isn't working exporter/splunkhec

Comments

@atoulme
Copy link
Contributor

atoulme commented Nov 29, 2023

Component(s)

No response

What happened?

Description

We see errors when sending HEC events with an event field set to a null value. We should always replace nil with an empty string to work around this limitation.

Collector version

0.89.0

Environment information

Environment

OS: (e.g., "Ubuntu 20.04")
Compiler(if manually compiled): (e.g., "go 14.2")

OpenTelemetry Collector configuration

No response

Log output

No response

Additional context

No response
@atoulme atoulme added bug Something isn't working needs triage New item requiring triage labels Nov 29, 2023
This was referenced Nov 29, 2023
Closed
Merged
@github-actions GitHub Actions
Copy link
Contributor

Pinging code owners for exporter/splunkhec: @atoulme @dmitryax. See Adding Labels via Comments if you do not have permissions to add labels yourself.

@crobert-1
Copy link
Member

Removing needs triage as a PR has been opened by the code owner that resolves this issue.

@crobert-1 crobert-1 removed the needs triage New item requiring triage label Nov 29, 2023
dmitryax pushed a commit that referenced this issue Dec 4, 2023
@atoulme
[exporter/splunkhec] do not export HEC events with a nil event field …
3d0c199 
…value (#29552)

**Description:**
Fixing a bug where Splunk rejects malformed HEC events with a nil event
field.

**Link to tracking Issue:**
Fixes #29551
@github-actions github-actions bot mentioned this issue Dec 5, 2023
Closed
@crobert-1 crobert-1 mentioned this issue Dec 7, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@atoulme atoulme

Labels
bug Something isn't working exporter/splunkhec
Projects
None yet
Milestone
No milestone
2 participants
@atoulme @crobert-1