jboss-fuse / fuse-apicurito-generator

Fuse Apicurito Generator
5 stars 14 forks source link

fix: use constant instead of simple expression #35

Closed zregvart closed 5 years ago

zregvart commented 5 years ago

By using simple expression the resource resource:classpath:openapi.json is loaded and then reinterpreted by the simple language expression support. This will lead to any escape newline characters within quoted strings to be expanded into newline characters breaking the string over multiple lines and in the end making the resulting resource served from /openapi.json an invalid JSON.

This is especially sensitive to example property within a definitions entry which are provided by end users when designing the API in Apicurio editor.

zregvart commented 5 years ago

The build seems to be referencing a hostname only accessible on the internal Red Hat network:

Could not transfer artifact io.fabric8:fabric8-maven-plugin:pom:3.5.33.fuse-730017 from/to jb-eap-7.1-brew (http://download.eng.bos.redhat.com/brewroot/repos/jb-eap-7.1-rhel-7-maven-build/latest/maven): download.eng.bos.redhat.com: Name or service not known: Unknown host download.eng.bos.redhat.com: Name or service not known
EricWittmann commented 5 years ago

@zregvart Try to rebase and build locally now. I think it should work.