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

Register fields of JAX-RS providers if necessary #20206

Merged
merged 1 commit into from
Sep 17, 2021
Merged

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Sep 16, 2021

Fixes: #20161

@quarkus-bot
Copy link

quarkus-bot bot commented Sep 16, 2021

This workflow status is outdated as a new workflow run has been triggered.

Failing Jobs - Building d4dc54f

Status Name Step Failures Logs Raw logs
Gradle Tests - JDK 11 Windows Build Failures Logs Raw logs
JVM Tests - JDK 11 Build Failures Logs Raw logs
✔️ JVM Tests - JDK 16
Native Tests - Misc4 Build Failures Logs Raw logs

Full information is available in the Build summary check run.

Failures

⚙️ Gradle Tests - JDK 11 Windows #

- Failing: integration-tests/gradle 

📦 integration-tests/gradle

io.quarkus.gradle.devmode.MultiModuleIncludedBuildTest.main line 24 - More details - Source on GitHub

java.lang.AssertionError: 

Expecting actual:

⚙️ JVM Tests - JDK 11 #

- Failing: integration-tests/elytron-undertow 

📦 integration-tests/elytron-undertow

io.quarkus.it.undertow.elytron.BaseAuthTest.testPost line 27 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <500>.

⚙️ Native Tests - Misc4 #

- Failing: integration-tests/gradle 

📦 integration-tests/gradle

io.quarkus.gradle.nativeimage.CustomNativeTestSourceSetIT.runNativeTests line 18 - More details - Source on GitHub

org.opentest4j.AssertionFailedError: 

expected: "SUCCESS"

@geoand geoand merged commit f982f5b into quarkusio:main Sep 17, 2021
@quarkus-bot quarkus-bot bot added this to the 2.3 - main milestone Sep 17, 2021
@geoand geoand deleted the #20161 branch September 17, 2021 06:05
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Sep 17, 2021
@gsmet gsmet modified the milestones: 2.3.0.CR1, 2.2.4.Final Nov 30, 2021
@fedinskiy
Copy link
Contributor

fedinskiy commented Jan 25, 2022

@gsmet this PR and linked issue are both marked with 2.2.4.Final milestone, yet these changes are not in 2.2 branch[1] and, as I understand it, they are not in 2.2.5.Final release. Is that correct?

[1]

// register the providers for reflection
for (String providerToRegister : jaxrsProvidersToRegisterBuildItem.getProviders()) {
reflectiveClass.produce(new ReflectiveClassBuildItem(false, false, providerToRegister));
}

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.

RestEasy SPI HttpRequest injection in ContainerRequestFilter fails silently in native image
4 participants