New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
parameter builder uses a comma instead of an empty string #1708
Comments
This is not a valid OpenAPI definition because path parameters must be required. If you replace |
Hi @encodeering, @hkosova is right about the validity of your definition. Though you're right that this is a possible Here is a POC demonstrating when the '/openapi/specification{plural}'.split('{plural}').join(undefined) As |
Hi @hkosova thanks for pointing this out, wasn't aware of this fact. Going to rewrite the specification as it may suddenly stop working if we update the dependencies. Cheers |
Hello,
it seems that an additional
,
can appended to the path in some cases and it has been detected using swagger ui's execution feature ([email protected]). It could be traced back to the following line:swagger-js/src/execute/oas3/parameter-builders.js
Line 26 in 3627ead
The default behavior of Array.join will be used if
styledValue
isundefined
.The following yaml definition creates this case, if the input field in swagger ui remains untouched
Many thanks in advance.
The text was updated successfully, but these errors were encountered: