Fix array in query params

[lan-tripla]

issue: https://stackoverflow.com/questions/68452127/pass-array-as-query-parameter-in-swagger-openapi-3-0

[exoego]

Could you add test for this

[lan-tripla]

Could you add test for this

updated test, could you check?

[exoego]

@lan-tripla
Sorry for very late response.

I am skeptic to this change, since it is not mandatory to name array-type query parameters like foo[] in OpenAPI 3.0, IIUC.
Even an OpenAPI 3.0 example from https://editor.swagger.io/ shows the example of name without []:

  /pet/findByStatus:
    get:
      tags:
        - pet
      summary: Finds Pets by status
      description: Multiple status values can be provided with comma separated strings
      operationId: findPetsByStatus
      parameters:
        - name: status
          in: query
          description: Status values that need to be considered for filter
          required: false
          explode: true
          schema:
            type: string
            default: available
            enum:
              - available
              - pending
              - sold

The stackoverflow issue you cited appears very specific issue to node-express's implementation for OpenAPI support.

I am fine to accept this change if a new option is added to name query params with [] and the option defaults to disabled for compatibility