Skip to content

Filter out "Transfer-Encoding" header from a REST Response #49894

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
Merged
merged 2 commits into from
Sep 5, 2025
Merged

Filter out "Transfer-Encoding" header from a REST Response #49894

merged 2 commits into from
Sep 5, 2025

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Sep 5, 2025

This header is set automatically by Vert.x depending
on whether the response was actually chunked,
so we need to make sure that is not taken into
account when processing Response or RestResponse

@geoand geoand requested a review from gastaldi September 5, 2025 06:58
@quarkus-bot quarkus-bot bot added the area/rest label Sep 5, 2025
@geoand geoand mentioned this pull request Sep 5, 2025
Closed
@suchwerk suchwerk mentioned this pull request Sep 5, 2025
Open
@quarkus-bot quarkus-bot

This comment has been minimized.

Sign in to view
@geoand
Filter out "Transfer-Encoding" header from a REST Response
96f2665 
This header is set automatically by Vert.x depending
on whether the response was actually chunked,
so we need to make sure that is not taken into
account when processing Response or RestResponse

Fixes: quarkusio#49875
@quarkus-bot quarkus-bot
Copy link

quarkus-bot bot commented Sep 5, 2025 •
edited by github-actions bot
Loading

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 96f2665.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

@gastaldi gastaldi merged commit ca1a5b6 into quarkusio:main Sep 5, 2025
52 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.28 - main milestone Sep 5, 2025
@gastaldi gastaldi deleted the #49875 branch September 5, 2025 11:50
@gsmet gsmet modified the milestones: 3.28 - main, 3.26.3 Sep 5, 2025
@geoand geoand mentioned this pull request Sep 8, 2025
Closed
@fedinskiy fedinskiy mentioned this pull request Sep 8, 2025
Open
@michalvavrik michalvavrik mentioned this pull request Sep 11, 2025
Merged
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gastaldi gastaldi gastaldi approved these changes

@FroMage FroMage FroMage approved these changes

Assignees
No one assigned
Labels
area/rest kind/bugfix
Projects
None yet
Milestone
3.26.3
Development

Successfully merging this pull request may close these issues.

Duplicate Transfer-Encoding: chunked headers in Quarkus response
4 participants
@geoand @gastaldi @FroMage @gsmet