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

Referenced RelaxNG schema not found via XML catalog #804

Closed
jperterm opened this issue Nov 15, 2022 · 5 comments · Fixed by eclipse/lemminx#1371
Closed

Referenced RelaxNG schema not found via XML catalog #804

jperterm opened this issue Nov 15, 2022 · 5 comments · Fixed by eclipse/lemminx#1371
Labels
bug Something isn't working relaxng
Milestone

Comments

@jperterm
Copy link

I tried out your new RelaxNG validation feature and it seems that an URI given in <?xml-model href="[...]" does not resolve against XML catalogs which can be configured via xml.catalogs setting. Absolute file URIs and OS path's are working fine.

[...]
<?xml-model href="http://canonical-uri/schema.rng" type="application/xml" schematypens="http://relaxng.org/ns/structure/1.0"?>
[...]

I tested with v0.22 and v0.23.2022111503 (pre-release).

@angelozerr
Copy link
Contributor

Indeed we don't use XML catalog for xml-model, we should implement that.

@angelozerr
Copy link
Contributor

@jperterm could you share the XML that you try to bind. I would like to test with your same rng URL.

Thanks

@jperterm
Copy link
Author

@angelozerr I made a simple test set - see test.zip, it contains the following files:

  • test.xml
  • schema.rng
  • catalog.xml

@angelozerr
Copy link
Contributor

Thanks so much! Let me investigate the problem.

@angelozerr angelozerr added relaxng bug Something isn't working labels Nov 15, 2022
angelozerr pushed a commit to angelozerr/lemminx that referenced this issue Nov 16, 2022
@angelozerr
Copy link
Contributor

angelozerr commented Nov 16, 2022

@jperterm I created a PR to fix the problem at eclipse/lemminx#1371 (comment)

@datho7561 datho7561 added this to the 0.23.0 milestone Nov 16, 2022
datho7561 pushed a commit to eclipse/lemminx that referenced this issue Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working relaxng
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants