-
Notifications
You must be signed in to change notification settings - Fork 20
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
OpenAPI 3.0.1 : path parameter validation does not comply the required schema #78
Comments
@daveshanley - can you please provide your comment on this ? My understanding is even if I dont specifiy a minLength property and keep required : true, I should not be allowed to make the call "/path123/ " . The validator should fail this. |
Hi, You have hit a block of code that has yet to be implemented. https://github.com/pb33f/libopenapi-validator/blob/main/parameters/path_parameters.go#L88 You can see the |
@daveshanley - thanks Dave . Any ETA for this ? Ideally, This is a very basic use case which should be handled by a validator. |
It's not on my immediate radar, please feel welcome to submit a PR! |
For a path parameter which is marked as required , empty value should not be allowed. Ideally , in this case , we do not need to proceed towards enum check as the absence of a value should violate the mandatory path parameter requirement criteria irrespective of the schema of path parameter
@daveshanley - raised a PR for this issue -#82 |
For a path parameter which is marked as required , empty value should not be allowed. Ideally , in this case , we do not need to proceed towards enum check as the absence of a value should violate the mandatory path parameter requirement criteria irrespective of the schema of path parameter
Added in |
Hi, I have the following schema defined.
(1) /path123/de path gives the following error which is expected.
"Path /path123/de is invalid or not supported"
(2) However if we pass empty path parameter /path123/ it passes and does not the return error
The text was updated successfully, but these errors were encountered: