Michael Edgar
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...