-
|
I'm trying to integrate the vert.x OpenAPI Router into a Quarkus Native app. I obtain the router object and mount the result of the OpenAPI as sub router. // Paraphrasing here
public void init(@Observes Router rootRouter) {
OpenAPIContract.from(vertx, spec)
.onSuccess(contract -> {
Router router = RouterBuilder.create(vertx, contract).createRouter()
rootRouter.route("/api").subRouter(router);
});
}Works like a charm. Until I tried to pack it in a native container with jib and quarkus-micro-image:2.0. When I run it, it fails with an error: Interestingly https://son-schema.org/draft/2020-12/schema is a working URL. So it seems the OpenAPI router wants to call out and can't from the container? What do I miss? |
Beta Was this translation helpful? Give feedback.
Replies: 3 comments 15 replies
-
|
/cc @EricWittmann (openapi), @MikeEdgar (openapi), @geoand (jib), @phillip-kruger (openapi), @zakkak (native-image) |
Beta Was this translation helpful? Give feedback.
-
|
Update: the issue seems with the native executable, When I start the runner in target, I get the same error: |
Beta Was this translation helpful? Give feedback.
-
|
Building a classic Java container works (It's just 3x bigger). Must be something in the lib that trips mandrel |
Beta Was this translation helpful? Give feedback.
-
|
I had a look and the problem is that you need to have the files in the That's why we have Quarkus extensions: we tweak the native image building configuration. Given we don't have an extension for this component, you need to tweak it yourself. See https://quarkus.io/guides/writing-native-applications-tips#including-resources to know how you can easily include additional resources with a pattern. Just include anything in the Note that it will solve this particular issue, there might be others. |
Beta Was this translation helpful? Give feedback.
-
|
I would think the former. |
Beta Was this translation helpful? Give feedback.
-
|
Something is off in my setup. I tried:
Verified that the values were picked up: But they don't get picked up. I also checked with a file in resources - same issue, doesn't get picked up |
Beta Was this translation helpful? Give feedback.
-
|
Created a reproducer. http://localhost:8080/hello/world works in |
Beta Was this translation helpful? Give feedback.
-
|
@Stwissel this seems related to the glob pattern being passed to In the reproducer Now my understanding is that |
Beta Was this translation helpful? Give feedback.
-
|
I see. So what would be the string for the original problem of including json-schema.org/** ? |
Beta Was this translation helpful? Give feedback.
-
|
FTR I created #44678 to fix the @Stwissel I don't see anything wrong with |
Beta Was this translation helpful? Give feedback.
-
|
Hmm. with the first expression fixed, the problem with json-schema went away. |
Beta Was this translation helpful? Give feedback.
-
I don't see anything new pushed there, perhaps you forgot to |
Beta Was this translation helpful? Give feedback.
@Stwissel this seems related to the glob pattern being passed to
quarkus.native.resources.includesIn the reproducer
**/*.jsongets translated to the regex.*/[^/]*\\.json, which doesn't matchtest.jsonbecause it needs at least one/. If you remove the/and pass***.jsonit will work.Now my understanding is that
**/*.jsonis expected to match alljsonfiles (see here), includingtest.json, which indicates thatquarkus/independent-projects/bootstrap/app-model/src/main/java/io/quarkus/util/GlobUtil.java
Line 65 in f959510