Skip to content
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

Add support for "XX" status code specifications #973

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

PSU3D0
Copy link

@PSU3D0 PSU3D0 commented Feb 20, 2024

As indicated in the OpenAPI specification, specifications may specify a range of response codes. See the following excerpt:

Any HTTP status code can be used as the property name, but only one property per code, to describe the expected response for that HTTP status code. This field MUST be enclosed in quotation marks (for example, "200") for compatibility between JSON and YAML. To define a range of response codes, this field MAY contain the uppercase wildcard character X. For example, 2XX represents all response codes between [200-299]. Only the following range definitions are allowed: 1XX, 2XX, 3XX, 4XX, and 5XX. If a response is defined using an explicit code, the explicit code definition takes precedence over the range definition for that code.


This was missing from the library. This PR fixes that. Cheers!

@dbanty
Copy link
Collaborator

dbanty commented Feb 20, 2024

Thanks! We'll probably want to add a new type with some template logic that can check ranges of status codes. I think as-is this will generate a ton of if statements

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.

2 participants