- marked as enhancement
Support validation for the parameters in cookie
Current library seems to only verify the parameter in header and query, if it is specified in cookie, there will not have any validation against it.
https://swagger.io/docs/specification/describing-parameters/#cookie-parameters
I think a validateCookieParameters
method need to be added to comply with specification.
Comments (8)
-
reporter -
reporter - changed title to Support validation for the parameters in cookie
-
Thanks for raising this.
You are correct - at the moment Cookie validation is not supported.
See https://bitbucket.org/atlassian/swagger-request-validator/src/master/docs/OPENAPIv3.md for the list what parts of the OpenAPI spec are currently supported.
The goal is (of course) to eventually get full coverage, but its going to be a matter of prioritization. If this is a blocker for you feel free to raise a PR and I’ll review it - always happy to accept contributions.
-
Hi James, thanks for your information. I will try to raise a PR in next 1 or 2 weeks.
Just to be sure, is this the guideline for contributing?
https://bitbucket.org/atlassian/swagger-request-validator/src/master/CONTRIBUTING.md
Signing CLA, adding tests, make sure code pass existing check.
-
A pull request has been raised : https://bitbucket.org/atlassian/swagger-request-validator/pull-requests/138
-
Awesome - thanks! I’ll take a look at it when I get a chance.
-
@James Navin Hi James, do you have a chance to take a look at the PR? Thanks!
-
- changed status to resolved
Available in v2.5.0
- Log in to comment