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 document is confusing a Resource Server (RS) with a Protected Resource #284

Closed
Denisthemalice opened this issue Jul 18, 2021 · 2 comments

Comments

@Denisthemalice
Copy link

The definition of a RS is given on page 7:

Resource Server (RS) server that provides operations on protected resources, where operations require a valid access token issued by an AS.

The definition of a Protected Resource is given on pages 8 and 9:

Protected Resource protected API (Application Programming Interface) served by an RS and that can be accessed by a client, if and only if a valid access token is provided.

Note: to avoid complex sentences, the specification document may simply refer to resource instead of protected resource.

When a client accesses to a RS for the first time, it does not necessarily know that a resource is protected using GNAP, and if protected by GNAP, which ASs are trusted by the RS.

Some resources on a RS can be protected using, e.g., an authentication mechanism while some other resources can be protected using GNAP or OAuth. The error codes to be returned by the server are not the same.

It would be worthwhile to delete the Note and within the document to make a difference between a RS and a PS.

@jricher
Copy link
Collaborator

jricher commented Jul 18, 2021

This document is not concerned with resources that are not protected by GNAP. It is not meant to be a universal security solution for the internet.

Closing this issue as this has been discussed several times.

@jricher jricher closed this as completed Jul 18, 2021
@aaronpk
Copy link
Collaborator

aaronpk commented Jul 19, 2021

See also: #212

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants