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

Update local openAPI schema #1066

Merged
merged 1 commit into from
Oct 21, 2024
Merged

Update local openAPI schema #1066

merged 1 commit into from
Oct 21, 2024

Conversation

cody-friedrichsen
Copy link
Contributor

@cody-friedrichsen cody-friedrichsen commented Oct 21, 2024

We want to retain the capability to attempt schema validation even when https://spec.openapis.org/oas/3.0/schema/2021-09-28 may be unavailable. This change pulls the latest iteration of the v3.0 schema from the openapi spec website and replaces the existing local schema.

  • formatting modified to use 4 space tabs instead of 8
  • top level description updated

This changes results in a full replace in the diff, but in reality the differences in the two files are only in the regex escape character count utilized. I generated a legible diff by piping both JSON files through JQ to standardize the formatting, then put those through diff. Here are the results

4c4
<   "description": "The description of OpenAPI v3.0.x documents, as defined by https://spec.openapis.org/oas/v3.0.3",
---
>   "description": "Validation schema for OpenAPI Specification 3.0.X.",
14c14
<       "pattern": "^3\\.0\\.\\d(-.+)?$"
---
>       "pattern": "^3\\\\.0\\\\.\\\\d(-.+)?$"
59c59
<         "^\\$ref$": {
---
>         "^\\\\$ref$": {
190c190
<             "^[a-zA-Z0-9\\.\\-_]+$": {
---
>             "^[a-zA-Z0-9\\\\.\\\\-_]+$": {
205c205
<             "^[a-zA-Z0-9\\.\\-_]+$": {
---
>             "^[a-zA-Z0-9\\\\.\\\\-_]+$": {
220c220
<             "^[a-zA-Z0-9\\.\\-_]+$": {
---
>             "^[a-zA-Z0-9\\\\.\\\\-_]+$": {
235c235
<             "^[a-zA-Z0-9\\.\\-_]+$": {
---
>             "^[a-zA-Z0-9\\\\.\\\\-_]+$": {
250c250
<             "^[a-zA-Z0-9\\.\\-_]+$": {
---
>             "^[a-zA-Z0-9\\\\.\\\\-_]+$": {
265c265
<             "^[a-zA-Z0-9\\.\\-_]+$": {
---
>             "^[a-zA-Z0-9\\\\.\\\\-_]+$": {
280c280
<             "^[a-zA-Z0-9\\.\\-_]+$": {
---
>             "^[a-zA-Z0-9\\\\.\\\\-_]+$": {
295c295
<             "^[a-zA-Z0-9\\.\\-_]+$": {
---
>             "^[a-zA-Z0-9\\\\.\\\\-_]+$": {
310c310
<             "^[a-zA-Z0-9\\.\\-_]+$": {
---
>             "^[a-zA-Z0-9\\\\.\\\\-_]+$": {
774c774
<         "^\\/": {
---
>         "^\\\\/": {
923c923
<         "^[1-5](?:\\d{2}|XX)$": {
---
>         "^[1-5](?:\\\\d{2}|XX)$": {

@cody-friedrichsen cody-friedrichsen merged commit 1b1d529 into master Oct 21, 2024
8 checks passed
@cody-friedrichsen cody-friedrichsen deleted the update_local_schema branch October 21, 2024 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants