Possible mistake in Context Processing Algorithm

Open
#518 5 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
35/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Stale
Domain
documentation

Research direction

Start with Step 5.5.1 of the Context Processing Algorithm and the compact-manifest test compact#tpr01. Compare the protected-term check against the example context and determine whether active context or result produces the specified behavior; done means the algorithm wording and the test expectation agree.

Written by the indexing model from the issue text.

Description

Errata spec:bug spec:substantive

Hi,

Step 5.5.1 of the Context Processing Algorithm says:

If override protected is false and active context contains any protected term definitions, an invalid context nullification has been detected and processing is aborted.

The verification is done on active context (the input context of the algorithm) and not result (the context being built). as far as I understand, this means that the following context, given in compact#tpr01 should be accepted:

{
  "@context": [
    {
      "@vocab": "http://example.com/",
      "@version": 1.1,
      "protected": {"@protected": true}
    },
    null
  ]
}

...but it is not. An invalid context nullification is expected to be returned. So my conclusion is that in Step 5.5.1 of the processing algorithm, result must be used to check the presence of protected term definitions, and not the original active context. Is that correct?

Dominant language
HTML
Stars
106
Forks
40
No merged PRs in 30d

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from w3c/json-ld-api

All issues in w3c/json-ld-api

Similar issues

More Documentation issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.