Skip to content

Conversation

@DCollart
Copy link
Contributor

@DCollart DCollart commented Oct 1, 2020

Fixes #523

For that new rule I used the "PathString" of the context in order to do the check.

I could also maybe have put that rule at the "Path" level, what do you think ?

@ghost
Copy link

ghost commented Oct 1, 2020

CLA assistant check
All CLA requirements met.

@darrelmiller
Copy link
Member

Doing the validation at the parameter level is better. There is an exception that was introduced in 3.1 that says if the path Item is empty, then the required parameter rule is not applied. https://github.com/OAI/OpenAPI-Specification/blob/master/versions/3.1.0.md#path-templating

@darrelmiller darrelmiller added this to the 1.2.4 milestone Oct 11, 2020
@darrelmiller darrelmiller modified the milestones: 1.2.4, 1.3 Feb 28, 2021
@darrelmiller darrelmiller changed the base branch from master to vnext March 14, 2021 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Path parameter not present in the path : No error

2 participants