Uploaded image for project: 'Camel'
  1. Camel
  2. CAMEL-15180

camel-restdsl-openapi-plugin NPE on optional field parameter.explode

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • 3.3.0
    • 3.4.0
    • camel-rest-openapi
    • None
    • Unknown

    Description

      Hey all, wanted to give the plugin a spin as we look to migrate some services from mule to camel and couldn't get the plugin to work for a simple project. Checked out plugin code and the tests are failing. I'll try to dig in and see what I can figure out but though I'd create an ticket for this.

      Here's the report from maven:
      [INFO] — maven-invoker-plugin:3.2.1:verify (integration-test) @ camel-restdsl-openapi-plugin —
      -[INFO] --------------------------------------------------
      [INFO] Build Summary:
      [INFO] Passed: 5, Failed: 6, Errors: 0, Skipped: 0
      -[INFO] --------------------------------------------------
      [ERROR] The following builds failed:
      [ERROR] * customized-v3/pom.xml
      [ERROR] * simple-dto-v3/pom.xml
      [ERROR] * simple-xml-dto-v3/pom.xml
      [ERROR] * customized/pom.xml
      [ERROR] * simple-dto/pom.xml
      [ERROR] * simple-xml-dto/pom.xml


      Edit:

      The failing integration tests were an error on my part when running the tests locally. However I found the cause for the original issue I was having where the plugin was not generating routes. I used the OpenAPI available here https://github.com/OAI/OpenAPI-Specification/blob/master/examples/v3.0/petstore-expanded.yaml The generator failed to parse it correctly because one of its parameters is of type `form` but does not explicitly set `explode` to anything so it results in a NPE. The spec states that `form` defaults to `true`

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              jam01 Jose Montoya
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: