Skip to content
This repository has been archived by the owner on Apr 5, 2023. It is now read-only.

rp adapter: docs - openid scope #375

Closed
llorllale opened this issue Oct 30, 2020 · 0 comments · Fixed by #376
Closed

rp adapter: docs - openid scope #375

llorllale opened this issue Oct 30, 2020 · 0 comments · Fixed by #376
Assignees
@llorllale
Labels
documentation Improvements or additions to documentation

Comments

@llorllale
Copy link
Contributor

Users may find it a little bit confusing that the openid scope does not need to be included in the call to register their client, yet they do have have to include it in the authorization request.

Need to add a note to make this clear. This example request is not including openid.

Related: #334
@llorllale llorllale added the documentation Improvements or additions to documentation label Oct 30, 2020
@llorllale llorllale self-assigned this Oct 30, 2020
llorllale pushed a commit to llorllale/edge-adapter that referenced this issue Oct 30, 2020
docs: trustbloc#375 rp-adapter include openid in authz request
5d20c79 
Signed-off-by: George Aristy <george.aristy@securekey.com>
@llorllale llorllale mentioned this issue Oct 30, 2020
Merged
rolsonquadras added a commit that referenced this issue Oct 30, 2020
@rolsonquadras
Merge pull request #376 from llorllale/375
ec0b8a5 
docs: #375 rp-adapter include `openid` in authz request
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@llorllale llorllale

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

docs: #375 rp-adapter include openid in authz request llorllale/edge-adapter
1 participant
@llorllale