Skip to content

fix(distributor): relax content-type check #10916

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

Merged
merged 1 commit into from
Mar 17, 2025
Merged

Conversation

krajorama
Copy link
Contributor

@krajorama krajorama commented Mar 17, 2025

What this PR does

We used to allow content type == text/plain for Prometheus remote write v1 push. Let's continue to not impact users.

Which issue(s) this PR fixes or relates to

Fixes : N/A (support case)
Follows: #10423

Checklist

  • Tests updated.
  • [N/A] Documentation added.
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX].
  • [N/A] about-versioning.md updated with experimental features.

@krajorama krajorama force-pushed the krajo/allow-malformed-rw1.0 branch 2 times, most recently from b420f24 to 02e3678 Compare March 17, 2025 14:19
We used to allow content type == text/plain for Prometheus
remote write v1 push. Let's continue to not impact users.

Signed-off-by: György Krajcsovits <[email protected]>
@krajorama krajorama force-pushed the krajo/allow-malformed-rw1.0 branch from 02e3678 to 7107521 Compare March 17, 2025 14:20
@krajorama krajorama marked this pull request as ready for review March 17, 2025 14:20
@krajorama krajorama requested a review from a team as a code owner March 17, 2025 14:20
@krajorama krajorama merged commit b569a46 into main Mar 17, 2025
28 checks passed
@krajorama krajorama deleted the krajo/allow-malformed-rw1.0 branch March 17, 2025 14:49
ying-jeanne pushed a commit that referenced this pull request Mar 19, 2025
We used to allow content type == text/plain for Prometheus
remote write v1 push. Let's continue to not impact users.

Signed-off-by: György Krajcsovits <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants