Michael Edgar

Results 220 comments of Michael Edgar

When you submit the request with `{"status":["foo","bar"]}` in Swagger UI, what does the generated `curl` command show was used as the URL for the request?

Thanks. Let me try some variations to see what works. There is also a good chance that since the documentation is so minimal (that I can see) on matrix parameters,...

Oh that's interesting, so Swagger UI isn't actually sending anything to the server with the old style?

@cristalp can you confirm that status serializes like `;status=["foo","bar"]` ? When I try it, Swagger UI generates `;status=foo,bar`. That is also incorrect, Swagger UI is not respecting the `explode: true`,...

+1 the scanner should know that reading a response example (or request body example) with a non-JSON media type should not attempt to parse the string.

@xfh , @cristalp - this should no longer be an issue as of version 4.0. Request bodies are marked as `required: true` by default (not due to the not-null constraint).

Since version 4.0 makes request bodies `required: true` by default, it no longer would make sense to set it as a result of `@NotNull`. I will go ahead and close...

To summarize, the issue is that `mp.openapi.scan.exclude` is not being read when the value is a list - but it is being read properly when it is a string?

This makes sense to me. Probably best to isolate it to the Spring module.

This will likely be in the [SpringParameterProcessor](https://github.com/smallrye/smallrye-open-api/blob/main/extension-spring/src/main/java/io/smallrye/openapi/spring/SpringParameterProcessor.java). I'm not entirely sure if `@ParameterObject` is analogous to `@BeanParam` or is it more like the MP OpenAPI `@Parameter` (i.e. documentation only, not...