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.

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

When validating an XML file with linked schema, Linter does not follow http redirects #1707

Open
Conal-Tuohy opened this issue Aug 5, 2020 · 1 comment
Open

When validating an XML file with linked schema, Linter does not follow http redirects #1707

Conal-Tuohy opened this issue Aug 5, 2020 · 1 comment
Labels
not-related-to-this-package

Comments

@Conal-Tuohy
Copy link

For example, the TEI consortium publishes HTTP URIs for standard TEI schemas, but when you resolve those URIs you are redirected to an HTTPS URI which then responds with the actual schema. The response to the HTTP URI is a "301" redirect with a message body containing HTML (saying "301 Moved Permanently").

What should happen is that the Linter should follow the 301 redirect and retrieve the actual schema.

What currently happens is that the Linter ignores the 301 response code, and attempts to interpret the HTML response body as a RelaxNG schema, which naturally fails.

Example XML document (which should validate according to the linked schemas):

<?xml version="1.0" encoding="UTF-8"?>
<?xml-model href="http://www.tei-c.org/release/xml/tei/custom/schema/relaxng/tei_xinclude.rng" type="application/xml" schematypens="http://relaxng.org/ns/structure/1.0"?>
<?xml-model href="http://www.tei-c.org/release/xml/tei/custom/schema/relaxng/tei_xinclude.rng" type="application/xml"
	schematypens="http://purl.oclc.org/dsdl/schematron"?>
<TEI xmlns:xi="http://www.w3.org/2001/XInclude" xmlns="http://www.tei-c.org/ns/1.0">
  <teiHeader>
      <fileDesc>
         <titleStmt>
            <title>Title</title>
         </titleStmt>
         <publicationStmt>
            <p>Publication Information</p>
         </publicationStmt>
         <sourceDesc>
            <p>Information about the source</p>
         </sourceDesc>
      </fileDesc>
  </teiHeader>
  <text>
      <body>
         <p>Some text here</p>
      </body>
  </text>
</TEI>

NB if you change the http scheme with https in those schema URIs, it does work.
@aminya
Copy link
Collaborator

aminya commented Jan 4, 2021

@Conal-Tuohy This should be done in the linter-* package that you use.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
not-related-to-this-package
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Conal-Tuohy @aminya