search

xproc / 3.0-test-suite

The 3.0 test suite.
https://test-suite.xproc.org/
MIT License
4 stars 3 forks source link

Propose change to base URI #733

Closed ndw closed 1 month ago

ndw commented 1 month ago

If you GET a document, I think the base URI of the document you "got" is the final URI after any redirects have been applied. So I think the right answer here is http**s**://...

ndw commented 1 month ago

Indeed. I wonder if p:document should raise an error if the return code indicates the document was not retrieved.

It might be worth keeping the test, but pointing ot a document that does exist.

ndw commented 1 month ago

Actually, the semantics of p:document defer to p:load which says:

It is a dynamic error (err:XD0011) if the resource referenced by a p:load element does not exist or cannot be accessed.

I think this test should raise an error. But I also think the spec should be clearer about that.

xml-project commented 1 month ago

Note on the original test: Resource "about.md" -> "about.html" was removed with https://github.com/xproc/xproc.org/pull/15 Since then github returned a html 404-page unnoticed.

ndw commented 1 month ago

Replaced by #738

Thank you @xml-project