-
Notifications
You must be signed in to change notification settings - Fork 192
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
Error while resolving allof schema: very confusing #417
Comments
@steve-nay-sage Thanks for reporting the issue and apologies for the late reply. It seems the third party module used for resolving |
Hi @VShingala , any updates on this issue? In postmanlabs/postman-app-support#11656 it says that this issue has been resolved.
ReDoc and Swagger edditor renders this yaml correctly |
Reopening since closed after related PR merge. Will close manually once it's included in release. |
@steve-nay-sage @Ifropc We've fixed this issue with the latest Postman app v10.22. I've confirmed both of your mentioned files are imported correctly. Feel free to reopen the issue in case you face any further related problem with it. |
Hi @VShingala, I am still experiencing an issue with an OAS json file (generated via NSwag) that is using allOf inside request and response objects. I don't actually get an error when using the command |
@redquis Thanks for reporting the issue! We're working on fix for it and it should be available in couple days. I'll update here once it gets released. |
@redquis This issue is now fixed with latest version v4.1.0 of openapi-to-postmanv2 module. It'll also gets fixed in postman app with upcoming releases. |
Thank you @VShingala, I can confirm that it works for me using v4.1.0 |
I've attached a fair-sized OAS file. (I had to change the file extension from .yaml to .txt or GitHub wouldn't accept it.) When I try this command on it:
It creates a Postman collection but gives this error:
I've tracked it down to the
key
property at line 2801 in the file, where if I addtype: string
then I don't get the error. However, there should be no reason to add that because the first object listed in the allOf already sets the type of that property to string. There are several other identical structures in the OAS file that do not cause the same error.To make things even more confusing, if I leave off either of the options at the end of the command, the error doesn't show up. So I can use
requestParametersResolution=Example
without seeing the error OR I can usefolderStategy=Tags
without seeing the error, but I can't use them both. That implies that there is nothing at all wrong with my OAS file, but some very odd problem that shows up when those two options are used together.It's beyond my skillset to debug further than that. I can point out the problem pretty specifically, but I can't dig into the code to find the source. I appreciate anyone who can look at this.
ap.openapi.txt
.
The text was updated successfully, but these errors were encountered: