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.

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

SimpleQuarkusRestTestCase#testFormMap sometimes fails with NPE #19471

Closed
gsmet opened this issue Aug 18, 2021 · 1 comment · Fixed by #19490
Closed

SimpleQuarkusRestTestCase#testFormMap sometimes fails with NPE #19471

gsmet opened this issue Aug 18, 2021 · 1 comment · Fixed by #19490
Labels
area/rest kind/bug Something isn't working
Milestone
2.1.3.Final

Comments

@gsmet
Copy link
Member

gsmet commented Aug 18, 2021

See the run for this PR: #19463

2021-08-17T19:37:29.9850663Z 2021-08-17 19:37:29,957 ERROR [io.qua.ver.htt.run.QuarkusErrorHandler] (executor-thread-0) HTTP Request to /simple/form-map failed, error id: e3672c38-a3f4-4013-876e-bdeec84f42f8-2: java.lang.NullPointerException
2021-08-17T19:37:29.9853878Z 	at org.jboss.resteasy.reactive.server.injection.ContextProducers.getTarget(ContextProducers.java:108)
2021-08-17T19:37:29.9857459Z 	at org.jboss.resteasy.reactive.server.injection.ContextProducers.resourceInfo(ContextProducers.java:90)
2021-08-17T19:37:29.9861602Z 	at org.jboss.resteasy.reactive.server.injection.ContextProducers_ProducerMethod_resourceInfo_be7f5503a87442251cde9fbc1d406fb350a529f0_Bean.create(ContextProducers_ProducerMethod_resourceInfo_be7f5503a87442251cde9fbc1d406fb350a529f0_Bean.zig:163)
2021-08-17T19:37:29.9866116Z 	at org.jboss.resteasy.reactive.server.injection.ContextProducers_ProducerMethod_resourceInfo_be7f5503a87442251cde9fbc1d406fb350a529f0_Bean.create(ContextProducers_ProducerMethod_resourceInfo_be7f5503a87442251cde9fbc1d406fb350a529f0_Bean.zig:194)
2021-08-17T19:37:29.9869160Z 	at io.quarkus.arc.impl.RequestContext.getIfActive(RequestContext.java:68)
2021-08-17T19:37:29.9870910Z 	at io.quarkus.arc.impl.ClientProxies.getDelegate(ClientProxies.java:33)
2021-08-17T19:37:29.9874045Z 	at org.jboss.resteasy.reactive.server.injection.ContextProducers_ProducerMethod_resourceInfo_be7f5503a87442251cde9fbc1d406fb350a529f0_ClientProxy.arc$delegate(ContextProducers_ProducerMethod_resourceInfo_be7f5503a87442251cde9fbc1d406fb350a529f0_ClientProxy.zig:60)
2021-08-17T19:37:29.9879272Z 	at org.jboss.resteasy.reactive.server.injection.ContextProducers_ProducerMethod_resourceInfo_be7f5503a87442251cde9fbc1d406fb350a529f0_ClientProxy.getResourceMethod(ContextProducers_ProducerMethod_resourceInfo_be7f5503a87442251cde9fbc1d406fb350a529f0_ClientProxy.zig:135)
2021-08-17T19:37:29.9884187Z 	at io.quarkus.resteasy.reactive.server.test.simple.ResourceInfoInjectingFilter.filter(ResourceInfoInjectingFilter.java:19)
2021-08-17T19:37:29.9888838Z 	at org.jboss.resteasy.reactive.server.handlers.ResourceRequestFilterHandler.handle(ResourceRequestFilterHandler.java:40)
2021-08-17T19:37:29.9893747Z 	at org.jboss.resteasy.reactive.server.handlers.ResourceRequestFilterHandler.handle(ResourceRequestFilterHandler.java:8)
2021-08-17T19:37:29.9898151Z 	at org.jboss.resteasy.reactive.common.core.AbstractResteasyReactiveContext.run(AbstractResteasyReactiveContext.java:133)
2021-08-17T19:37:29.9971999Z 	at io.quarkus.vertx.core.runtime.VertxCoreRecorder$13.runWith(VertxCoreRecorder.java:538)
2021-08-17T19:37:29.9974080Z 	at org.jboss.threads.EnhancedQueueExecutor$Task.run(EnhancedQueueExecutor.java:2449)
2021-08-17T19:37:29.9976066Z 	at org.jboss.threads.EnhancedQueueExecutor$ThreadBody.run(EnhancedQueueExecutor.java:1478)
2021-08-17T19:37:29.9977965Z 	at org.jboss.threads.DelegatingRunnable.run(DelegatingRunnable.java:29)
2021-08-17T19:37:29.9980149Z 	at org.jboss.threads.ThreadLocalResettingRunnable.run(ThreadLocalResettingRunnable.java:29)
2021-08-17T19:37:29.9982705Z 	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
2021-08-17T19:37:29.9984394Z 	at java.base/java.lang.Thread.run(Thread.java:829)
@quarkus-bot
Copy link

quarkus-bot bot commented Aug 18, 2021

/cc @FroMage, @geoand, @stuartwdouglas

@gsmet gsmet mentioned this issue Aug 18, 2021
Merged
stuartwdouglas added a commit to stuartwdouglas/quarkus that referenced this issue Aug 19, 2021
@stuartwdouglas
Fix RESTEasy Reactive race
b361dbe 
These executor dispatch has to be the last thing in the method, to
prevent a possible race

Fixes quarkusio#19471
@stuartwdouglas stuartwdouglas mentioned this issue Aug 19, 2021
Merged
@quarkus-bot quarkus-bot bot added this to the 2.3 - main milestone Aug 19, 2021
gsmet pushed a commit that referenced this issue Aug 19, 2021
@stuartwdouglas @gsmet
Fix RESTEasy Reactive race
58ff213 
These executor dispatch has to be the last thing in the method, to
prevent a possible race

Fixes #19471

(cherry picked from commit b361dbe)
@gsmet gsmet modified the milestones: 2.3 - main, 2.1.3.Final Aug 19, 2021
gsmet pushed a commit to gsmet/quarkus that referenced this issue Aug 23, 2021
@stuartwdouglas @gsmet
Fix RESTEasy Reactive race
180e859 
These executor dispatch has to be the last thing in the method, to
prevent a possible race

Fixes quarkusio#19471

(cherry picked from commit b361dbe)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/rest kind/bug Something isn't working
Projects
None yet
Milestone
2.1.3.Final
Development

Successfully merging a pull request may close this issue.

Fix RESTEasy Reactive race stuartwdouglas/quarkus
1 participant
@gsmet