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

RESTEasy Reactive - deploy failure when using opentracing #13981

Closed
bcluap opened this issue Dec 18, 2020 · 1 comment · Fixed by #13983
Closed

RESTEasy Reactive - deploy failure when using opentracing #13981

bcluap opened this issue Dec 18, 2020 · 1 comment · Fixed by #13983
Assignees
@geoand
Labels
area/rest area/smallrye area/tracing kind/bug Something isn't working
Milestone
1.11.0.Beta2

Comments

@bcluap
Copy link

bcluap commented Dec 18, 2020

Describe the bug
Build fails with IllegalStateException when using opentracing and a rest method which accepts a List of Strings
Expected behavior
Build should work
Actual behavior
Build fails when adding:

io.quarkus
quarkus-smallrye-opentracing

To Reproduce
git clone https://github.com/bcluap/quarkus-examples.git
cd quarkus-examples/resteasy-reactive
mvn clean test

Additional context
I believe line 29 of DynamicFeatureResourceInfo should be:

if (!parameterTypes[i].getName().equals(resourceMethod.getParameters()[i].declaredType)) {
@bcluap bcluap added the kind/bug Something isn't working label Dec 18, 2020
@ghost
Copy link

ghost commented Dec 18, 2020

/cc @FroMage, @Ladicek, @geoand, @kenfinnigan, @stuartwdouglas

@geoand geoand self-assigned this Dec 19, 2020
geoand added a commit to geoand/quarkus that referenced this issue Dec 19, 2020
@geoand
Replace useless DynamicFeatureResourceInfo class with ResteasyReactiv…
1854d37 
…eResourceInfo

Fixes: quarkusio#13981
geoand added a commit to geoand/quarkus that referenced this issue Dec 19, 2020
@geoand
Replace useless DynamicFeatureResourceInfo class with ResteasyReactiv…
011f723 
…eResourceInfo

Fixes: quarkusio#13981
@geoand geoand mentioned this issue Dec 19, 2020
Merged
@snowdrop-bot snowdrop-bot mentioned this issue Dec 19, 2020
Closed
gsmet added a commit that referenced this issue Dec 21, 2020
@gsmet
Merge pull request #13983 from geoand/#13981
796af8e 
Replace useless DynamicFeatureResourceInfo class with ResteasyReactiveResourceInfo
@ghost ghost added this to the 1.11 - master milestone Dec 21, 2020
cemnura pushed a commit to cemnura/quarkus that referenced this issue Dec 22, 2020
@geoand @cemnura
Replace useless DynamicFeatureResourceInfo class with ResteasyReactiv…
e98216a 
…eResourceInfo

Fixes: quarkusio#13981
renegrob pushed a commit to renegrob/quarkus that referenced this issue Dec 23, 2020
@geoand @renegrob
Replace useless DynamicFeatureResourceInfo class with ResteasyReactiv…
bafedf3 
…eResourceInfo

Fixes: quarkusio#13981
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@geoand geoand

Labels
area/rest area/smallrye area/tracing kind/bug Something isn't working
Projects
None yet
Milestone
1.11.0.Beta2
Development

Successfully merging a pull request may close this issue.

Replace useless DynamicFeatureResourceInfo class with ResteasyReactiveResourceInfo geoand/quarkus
2 participants
@geoand @bcluap