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

2.14.3 backports 1 #29650

Merged
merged 13 commits into from
Dec 5, 2022
Merged

2.14.3 backports 1 #29650

merged 13 commits into from
Dec 5, 2022

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Dec 2, 2022

Please don't merge, I will merge it myself.

michalvavrik and others added 13 commits December 2, 2022 19:08
In particular Maven users may miss that section about updating their .dockerignore file for in-container builds, which only are shown after the example Dockerfile for Gradle. By pulling this information before the two example Dockerfiles, everyone should read them.

(cherry picked from commit a26196c)
(cherry picked from commit c39f6cd)
(cherry picked from commit 7d88338)
(cherry picked from commit 5deeb18)
Fixes quarkusio#29431

This is a workaround though. The core of the problem is that when in
HTTPS, event.request().absoluteURI() returns an HTTP url, which is wrong
and leads to the redirect not working.

(cherry picked from commit e2572f4)
(cherry picked from commit 4a14018)
@gsmet gsmet added the area/infra internal and infrastructure related issues label Dec 2, 2022
@gsmet gsmet self-assigned this Dec 2, 2022
@quarkus-bot
Copy link

quarkus-bot bot commented Dec 2, 2022

Failing Jobs - Building 798f188

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

Full information is available in the Build summary check run.

Failures

⚙️ JVM Tests - JDK 11 Windows #

- Failing: extensions/resteasy-reactive/rest-client-reactive/deployment 
! Skipped: extensions/keycloak-admin-client-reactive/deployment extensions/oidc-client-reactive-filter/deployment extensions/oidc-token-propagation-reactive/deployment and 17 more

📦 extensions/resteasy-reactive/rest-client-reactive/deployment

io.quarkus.rest.client.reactive.timeout.BuilderReadTimeoutTest.shouldNotTimeoutOnFastResponse line 53 - More details - Source on GitHub

javax.ws.rs.ProcessingException: The timeout period of 1000ms has been exceeded while executing GET /fast for server localhost:8081
	at org.jboss.resteasy.reactive.client.impl.InvocationBuilderImpl.unwrap(InvocationBuilderImpl.java:223)
	at org.jboss.resteasy.reactive.client.impl.InvocationBuilderImpl.method(InvocationBuilderImpl.java:324)

@gsmet gsmet merged commit 702c00a into quarkusio:2.14 Dec 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/infra internal and infrastructure related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants