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

The usage of URL, URI and URI-reference is not aligned #114

Closed
e-backmark-ericsson opened this issue Feb 16, 2023 · 4 comments
Closed

The usage of URL, URI and URI-reference is not aligned #114

e-backmark-ericsson opened this issue Feb 16, 2023 · 4 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@e-backmark-ericsson
Copy link
Contributor

As noted in this comment, our usage of universal resource locators/references is not really aligned over our json schemas and markdown docs. We should decide on what format to use and then make sure it is updated in the relevant files

@e-backmark-ericsson e-backmark-ericsson added the bug Something isn't working label Feb 16, 2023
@e-backmark-ericsson e-backmark-ericsson added this to the v0.2 milestone Feb 16, 2023
This was referenced Feb 16, 2023
@afrittoli
Copy link
Contributor

I don't think there is any misalignment in the usage of URI and URI refs - however, the JSON schema does not enforce URI/URI-ref yet. This is only because I didn't realise it was possible in jsonschema - I will add it as part of this issue.

@afrittoli
Copy link
Contributor

Discussed this item in the WG:

  • URI-reference is a superset of URI
  • For now (v0.2) we may keep URI-reference and indicate in the spec that URI is preferred where available

@afrittoli
Copy link
Contributor

We already indicate URI as the recommended format:

source (context)

  • Type: [URI-Reference][typesystem]
    - Description: defines the context in which an event happened. The main purpose
    of the source is to provide global uniqueness for source +
    id.

    (...)

  • Constraints:
    - REQUIRED
    - MUST be a non-empty URI-reference
    - An absolute URI is RECOMMENDED

@afrittoli
Copy link
Contributor

Fixed by #116

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

No branches or pull requests

2 participants