Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

adoptACLDefault does not copy triples with acl:Authorization

See original GitHub issue

I’m writing tests for the acl module, and noticed that adoptACLDefault does not copy the triples containing acl:Authorization into the new graph it builds. It seems that NSS does not care about this triples though, which is why the bug has gone unnoticed.

I’m creating this issue to verify my suspicion with others. If it is the case that NSS should require acl:Authorization triples, I’ll create a bug there as well.

Issue Analytics

State:
Created 3 years ago
Comments:5 (5 by maintainers)

Top GitHub Comments

2reactions

megothcommented, Mar 23, 2020

I found the issue “Is acl:Authorization mandatory?” in the WAC spec, where the question is discussed as part of the new specification.

I’ll keep an eye out for those discussions, but until there’s a conclusion I’ll wait to create issues on NSS and the test-suite.

That being said, for the sake of semantics I suggest that we make sure that acl:Authorization is added as part of the code we touch in Solid UI and other places. (I’ve made sure to fix this in in adoptACLDefault in a PR that is coming very soon.)

2reactions

michielbdejongcommented, Mar 23, 2020

Actually a triple bug, Solid UI + NSS + test-suite. 😃