solid/authorization-panel

Remove acp:mode from Context properties

Closed this issue · 2 comments

timbl commented

On proposal

ACP

Details

In section 3, the Context structure includes an acp:mode property as "acp:mode:
The mode attribute describes requested modes of access."

But the system does not provide just one mode at a time, it provides a set of modes. That property on the context is not needed and should be removed.

In general, even when a read request comes in, the client typically needs to know (Through WAC-Allow etc) what other access is allowed.

Acceptance criteria

Remove acp:modes from the context each time it appears

matthieubosquet commented

Thank you for this @timbl! I think that you are right, the acp:mode property is superfluous.

I have created a PR addressing this issue that I hope we can merge soon. It is a in the list of topics for the authorization-panel next wednesday (28/09).

langsamu commented

I'd implemented the ACP access control resolution algorithm very recently.
I agree that acp:mode is not used anywhere and should be removed.

  • 👍1