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

Recommend the path-noscheme form for relative URIs. #468

Closed
ghost opened this issue Jul 27, 2020 · 1 comment
Closed

Recommend the path-noscheme form for relative URIs. #468

ghost opened this issue Jul 27, 2020 · 1 comment

Comments

@ghost
Copy link

ghost commented Jul 27, 2020

The reason for specifying a URI as a relative reference is to make it easy to "rebase" URIs when you move a SARIF file from one machine to another where the repo root isn't in the same place. You can't do that if the relative references start at the root of the drive, for example /a/b/c.txt.

Consider adding either a non-normative note or a SHOULD-level requirement for relative references to use the path-noscheme form (RFC 3986 §4.2), which is roughly a/b/c.txt (no leading slash).

@michaelcfanning michaelcfanning changed the title Recommend the path-noscheme form for relative URIs Recommend the path-noscheme form for relative URIs. Nov 8, 2021
@dmk42
Copy link
Contributor

dmk42 commented Dec 19, 2023

Now that the Errata document has been published, this has been fixed in the official version of the standard.

@dmk42 dmk42 closed this as completed Dec 19, 2023
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

2 participants