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.

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

return JSON ok response for hec receiver raw path #29875

Merged
merged 4 commits into from Dec 15, 2023

Conversation

splunkericl
Copy link
Contributor

Description:
Adds enhancement for splunk hec receiver raw path

  • previously it returns no response in http body: it nows returns json response

Link to tracking Issue:
#29745

Testing:

  • Updated unit tests.
  • Manually verify response is in json with postman

Documentation:
N/A

@dmitryax dmitryax merged commit ce75979 into open-telemetry:main Dec 15, 2023
85 checks passed
@github-actions github-actions bot added this to the next release milestone Dec 15, 2023
nslaughter pushed a commit to nslaughter/opentelemetry-collector-contrib that referenced this pull request Dec 27, 2023
**Description:**
Adds enhancement for splunk hec receiver raw path
- previously it returns no response in http body: it nows returns json
response

**Link to tracking Issue:**

open-telemetry#29745
nslaughter pushed a commit to nslaughter/opentelemetry-collector-contrib that referenced this pull request Dec 27, 2023
**Description:**
Adds enhancement for splunk hec receiver raw path
- previously it returns no response in http body: it nows returns json
response

**Link to tracking Issue:**

open-telemetry#29745
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Jan 10, 2024
**Description:**
Adds enhancement for splunk hec receiver raw path
- previously it returns no response in http body: it nows returns json
response

**Link to tracking Issue:**

open-telemetry#29745
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants