Parameter with an array is not being validated and maybe not parsed correctly

[mikestead]

Parameter with an array is not being validated and maybe not parsed correctly

[jamidon]

I've traced it down into swaggerSpec.js in the createParamGroupSchemas() method. This method assumes each parameter's schema has the 'properties' property whereas the schema for an array parameter doesn't.

I'm wondering if I were possible to flatten out the array definitions to remove the $ref's if the jsonschema validate method would work.

Thoughts?

[mikestead]

Thanks for the details. I'm on a short break right now but will take a look next week when I'm back.

[jamidon]

Thanks.

…

-- jda

On Jan 26, 2017 9:02 PM, "Mike Stead" ***@***.***> wrote: Thanks for the details. I'm on a short break right now but will take a look next week when I'm back. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#11 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ADbkJjtRXL22rH2u1W3s2rbfa3vZqvmnks5rWVAkgaJpZM4LuOwV> .

[jamidon]

I found a way around this issue, and I'm fairly certain it has nothing to do with swagger-routes but the jsonschema module.

If I change the definition of the parameter in the add method from:

        "parameters": [
          {
            "name": "organization",
            "in": "body",
            "description": "organization(s) to add to the system",
            "required": true,
            "schema": {
              "$ref": "#/definitions/NewOrganizations"
            }
          }
        ]

to

        "parameters": [
          {
            "name": "organization",
            "in": "body",
            "description": "organization(s) to add to the system",
            "required": true,
            "schema": {
              "type": "array",
              "items": {
                "$ref": "#/definitions/NewOrganization"
              }
            }
          }
        ],

everything works as expected. Essentially the difference in the two definitions is the second one is a bit more explicit but otherwise seems equivalent to me.

Anyway, sorry for the noise.

[mikestead]

Good to hear, thanks for posting the solution 👍