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.

Sign up for GitHub

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

Jump to bottom

fix: do not resolve meta refs #6

Open

fhueser wants to merge 1 commit into Eomm:main from fhueser:dont-resolve-meta-ref

fhueser commented Aug 18, 2022

These changes allow for meta refs to be skipped while resolving a schema which is necessary to conform to https://json-schema.org/draft-07/schema.

{
  $id: 'schemaWithMetaRef',
  type: 'object',
  properties: {
    $ref: { type: 'string' }
  }
}


          fix: do not resolve meta refs

3c16d35

Owner

Eomm commented Aug 21, 2022

Could you elaborate on this use case?

By reading the spec: https://datatracker.ietf.org/doc/html/draft-handrews-json-schema-01#section-8.3

An object schema with a "$ref" property MUST be interpreted as a
"$ref" reference. The value of the "$ref" property MUST be a URI
Reference. Resolved against the current URI base, it identifies the
URI of a schema to use. All other properties in a "$ref" object MUST
be ignored.

I understand that $ref cannot contain an object.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet