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

Remove Swagger ModelResolverSubstitutions #6593

Open
jamesnetherton opened this issue Oct 4, 2024 · 3 comments
Open

Remove Swagger ModelResolverSubstitutions #6593

jamesnetherton opened this issue Oct 4, 2024 · 3 comments

Comments

@jamesnetherton
Copy link
Contributor

Bug description

Quarkus has upgraded Jackson to 2.18.0 & Swagger Core is using some deprecated APIs which got removed.

Caused by: com.oracle.graal.pointsto.constraints.UnsupportedFeatureException: Discovered unresolved method during parsing: com.fasterxml.jackson.databind.BeanDescription.findJsonValueMethod(). This error is reported at image build time because class io.swagger.v3.core.jackson.ModelResolver is registered for linking at image build time by command line and command line.
Error encountered while parsing io.swagger.v3.core.jackson.ModelResolver.findJsonValueType(ModelResolver.java:1065)
Parsing context:
   at io.swagger.v3.core.jackson.ModelResolver.resolve(ModelResolver.java:410)
   at io.swagger.v3.core.converter.ModelConverterContextImpl.resolve(ModelConverterContextImpl.java:97)
   at io.swagger.v3.core.converter.ModelConverters.readAll(ModelConverters.java:137)
   at io.swagger.v3.core.converter.ModelConverters.readAll(ModelConverters.java:128)
   at org.apache.camel.openapi.RestModelConverters.readClassOpenApi3(RestModelConverters.java:84)
   at org.apache.camel.openapi.RestModelConverters.readClass(RestModelConverters.java:73)
   at org.apache.camel.openapi.RestOpenApiReader.appendModels(RestOpenApiReader.java:1028)
   at org.apache.camel.openapi.RestOpenApiReader.parse(RestOpenApiReader.java:324)
   at org.apache.camel.openapi.RestOpenApiReader.read(RestOpenApiReader.java:209)
   at org.apache.camel.openapi.RestOpenApiSupport.renderResourceListing(RestOpenApiSupport.java:315)
   at org.apache.camel.openapi.RestOpenApiProcessor.doInit(RestOpenApiProcessor.java:75)
   at org.apache.camel.support.service.BaseService.init(BaseService.java:85)
   at org.apache.camel.support.service.BaseService.start(BaseService.java:115)
   at org.apache.camel.support.ScheduledPollConsumer.beforePoll(ScheduledPollConsumer.java:736)
   at root method.(Unknown Source)
@jamesnetherton jamesnetherton added the bug Something isn't working label Oct 4, 2024
@jamesnetherton
Copy link
Contributor Author

Some more detail in swagger-api/swagger-core#4755.

@jamesnetherton
Copy link
Contributor Author

Fixed in #6648.

@jamesnetherton jamesnetherton changed the title [quarkus-main] openapi-java integration tests native build fails due to unresolved method openapi-java integration tests native build fails due to unresolved method Oct 16, 2024
@jamesnetherton jamesnetherton added this to the 3.16.0 milestone Oct 16, 2024
@jamesnetherton jamesnetherton removed this from the 3.16.0 milestone Oct 16, 2024
@jamesnetherton
Copy link
Contributor Author

Reopening since we should remove the workaround when we get an upgraded swagger-core.

@jamesnetherton jamesnetherton changed the title openapi-java integration tests native build fails due to unresolved method Remove Swagger ModelResolverSubstitutions Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant