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

Editorial: DCQL integration into Section 5 #364

Open
dbluhm opened this issue Dec 4, 2024 · 0 comments
Open

Editorial: DCQL integration into Section 5 #364

dbluhm opened this issue Dec 4, 2024 · 0 comments
Milestone

Comments

@dbluhm
Copy link

dbluhm commented Dec 4, 2024

While reading through the latest drafts after the introduction of DCQL, we noticed a few places where DCQL could be better integrated into text that existed prior to the introduction.

  • Section 5, Paragraph 4 specifically mentions PE, giving an overview of how it's used to request credentials, but fails to mention DCQL
  • In Section 5, the lack of a subsection for DCQL makes it easy to overlook that dcql_query is an expected parameter in the Authorization Request.
  • Section 5.3 shows an example authorization request with a presenation_definition parameter but not one with dcql_query.
  • Section 5.6, Paragrah 2 contains a typo resulting in a bulleted list not getting rendered correctly.
  • Section 5.6, Paragraph 4 and Paragraph 5, which suggest possible options for where well-known scope values could be defined, specifically discusses PE but not DCQL.
@Sakurann Sakurann added this to the 1.1 milestone Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants