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

Ensure that Transfer-Encoding and Content-Length are not both set #29460

Merged
merged 1 commit into from Nov 24, 2022

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Nov 24, 2022

The combination of these two HTTP headers is illegal, so let's make sure RESTEasy Reactive does not send the Content-Length header when Transfer-Encoding is set

Fixes: #29059

The combination of these two HTTP headers is illegal, so
let's make sure RESTEasy Reactive does not send the Content-Length
header when Transfer-Encoding is set

Fixes: quarkusio#29059
@geoand geoand added triage/waiting-for-ci Ready to merge when CI successfully finishes triage/backport-2.13 triage/backport? labels Nov 24, 2022
@quarkus-bot
Copy link

quarkus-bot bot commented Nov 24, 2022

Failing Jobs - Building b632c9e

Status Name Step Failures Logs Raw logs
✔️ JVM Tests - JDK 11
JVM Tests - JDK 17 Build Failures Logs Raw logs
✔️ JVM Tests - JDK 17 MacOS M1
✔️ JVM Tests - JDK 18

Full information is available in the Build summary check run.

Failures

⚙️ JVM Tests - JDK 17 #

- Failing: extensions/micrometer/deployment integration-tests/reactive-messaging-hibernate-orm 
! Skipped: extensions/micrometer-registry-prometheus/deployment extensions/quartz/deployment extensions/scheduler/deployment and 18 more

📦 extensions/micrometer/deployment

io.quarkus.micrometer.deployment.binder.HttpDevModeConfigTest.test line 90 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Response body doesn't match expectation.

📦 integration-tests/reactive-messaging-hibernate-orm

io.quarkus.it.kafka.KafkaConnectorTest.testFruits line 50 - More details - Source on GitHub

org.awaitility.core.ConditionTimeoutException: Assertion condition defined as a io.quarkus.it.kafka.KafkaConnectorTest expected: <6> but was: <1> within 10 seconds.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
	at org.awaitility.core.AssertionCondition.await(AssertionCondition.java:119)

@geoand geoand merged commit 5911db1 into quarkusio:main Nov 24, 2022
@quarkus-bot quarkus-bot bot added this to the 2.15 - main milestone Nov 24, 2022
@geoand geoand deleted the #29059 branch November 24, 2022 10:47
@quarkus-bot quarkus-bot bot added kind/bugfix and removed triage/waiting-for-ci Ready to merge when CI successfully finishes labels Nov 24, 2022
@gsmet gsmet modified the milestones: 2.15 - main, 2.14.2.Final Nov 24, 2022
@gsmet gsmet modified the milestones: 2.14.2.Final, 2.13.6.Final Dec 14, 2022
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.

Ensure that content-length is not set when transfer-encoding exists while propogating headers from rest client
3 participants