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

Improve external reference resolution #223

Open
ewoutkramer opened this issue Dec 11, 2023 · 0 comments
Open

Improve external reference resolution #223

ewoutkramer opened this issue Dec 11, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@ewoutkramer
Copy link
Member

We can currently have two choices when resolving an external reference to a resource:

  • Disable external reference resolution by not providing a resolver. In this case you won't get any errors or warnings.
  • Provide a resolver. In this case you either succesfully resolve or else an error is produced by the validator.

It is clear to me that implementers may wish to have behaviours where a set of resources do not need to be resolved, or where we only want to resolve a given set. In this case you may only wish to produce a warning, or maybe nothing at all if the resources is outside the set. This is currently not possible.

I suggest we change the delegate so it returns an enum with the desired error behaviour (just like for extensions) + an out parameter that contains the resolved resource (if any). The delegate than has full control over the behaviour.

@ewoutkramer ewoutkramer added the enhancement New feature or request label Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Todo
Development

No branches or pull requests

1 participant