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

Resolving IRIs does not always emit IRI warnings. #2922

Closed
afs opened this issue Jan 6, 2025 · 2 comments
Closed

Resolving IRIs does not always emit IRI warnings. #2922

afs opened this issue Jan 6, 2025 · 2 comments
Assignees

Comments

@afs
Copy link
Member

afs commented Jan 6, 2025

Version

5.2.0

What happened?

riot on

{
    "@context": {
        "blob": "vocab:blob:"
    },
    "blob:at": {
        "@id": "file:/pdf/2015.pdf"
    }
}

gives no warning,

But riot on

_:B42f7628296b620abed4ec598d76b4cf1 <dokk:vocab:blob:at> <file:/pdf/2015.pdf> .

does.,

Are you interested in making a pull request?

Yes

@afs afs self-assigned this Jan 6, 2025
@afs afs changed the title Parsing JSON-LD does not emit IRI warnings. Resolving IRIs does not always emit IRI warnings. Jan 7, 2025
@afs
Copy link
Member Author

afs commented Jan 7, 2025

This is not JSON-LD speciifc.
Title changed.

When the base URI and the URI being resolved both a URI scheme and it is the same scheme, the parsing of the input URI (to be resolved) is not checked for warning. Only errors and syntax errors.

@afs afs added the IRI label Jan 11, 2025
@afs
Copy link
Member Author

afs commented Jan 11, 2025

On further investigation, this isn't a bug. It is just the way that it plays out.

Jena is following RFC 3986 reference resolution (the "non-strict" version) in both jena-iri and jena-iri3986. This means that file: needs to be file:/// (empty host, not missing host).

@afs afs closed this as completed Jan 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant