You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@camel.apache.org by "Claus Ibsen (Jira)" <ji...@apache.org> on 2020/06/10 08:38:00 UTC

[jira] [Commented] (CAMEL-15143) Remove jaxb requirement from camel-openapi-java

    [ https://issues.apache.org/jira/browse/CAMEL-15143?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17130407#comment-17130407 ] 

Claus Ibsen commented on CAMEL-15143:
-------------------------------------

So the problem is that we dont know if the rest model are using property placeholders {{foo}} which we dont want to output in the openapi doc, but instead replace those with their actual values.

So instead of using JAXB and dump as XML (xml-io do not support model -> xml) we could do a old regexp search/replace for {{ }} tokens and replace with resolved properties from the properties component.

> Remove jaxb requirement from camel-openapi-java
> -----------------------------------------------
>
>                 Key: CAMEL-15143
>                 URL: https://issues.apache.org/jira/browse/CAMEL-15143
>             Project: Camel
>          Issue Type: Improvement
>          Components: camel-openapi-java, camel-swagger-java
>            Reporter: Luca Burgazzoli
>            Assignee: Claus Ibsen
>            Priority: Major
>             Fix For: 3.4.0
>
>
> The camel-openapi-java and camel-swagger-java requires to dump rest definitions to xml and then re-parse the definition: 
> - https://github.com/apache/camel/blob/35eea9156d9828363825de3a12cc2ebd8dfeef27/components/camel-openapi-java/src/main/java/org/apache/camel/openapi/RestOpenApiSupport.java#L195-L237
> - https://github.com/apache/camel/blob/35eea9156d9828363825de3a12cc2ebd8dfeef27/components/camel-swagger-java/src/main/java/org/apache/camel/swagger/RestSwaggerSupport.java#L147-L189
> This seems to be overkilling and requires to bring jaxb to the game which has negative impact on camel-quarkus



--
This message was sent by Atlassian Jira
(v8.3.4#803005)