openid / openid/OpenID4VP

Actionable references to security BCP

Open
#147 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

security
Dominant language
Shell
Stars
112
Forks
38
Avg merge
12d 19h
Merged PRs (30d)
4

Description

The draft currently contains the following references to the OAuth Security BCP:

(1)

Any of the OAuth 2.0 related specifications, such as [RFC9126] and [RFC9101], and Best Current Practice (BCP) documents, such as [RFC8252] and [I-D.ietf-oauth-security-topics], can be implemented on top of this specification.

(2)

The Authorization Request follows the definition given in [RFC6749] taking into account the recommendations given in [I-D.ietf-oauth-security-topics].

(3)

The state parameter defined in Section 4.1.1 of [RFC6749] may be used by a verifier to link requests and responses. Also see Section 3.6 and Section 5.3.5 of [RFC6819], and [I-D.ietf-oauth-security-topics].

(4)

The Wallet MUST ensure the data in the Authorization Response cannot leak through Response URIs. When using pre-registered Response URIs, the Wallet MUST comply with best practices for redirect URI validation as defined in [I-D.ietf-oauth-security-topics].

For (1), I think the layering should be the other way round: This draft can be implemented while following the security BCP.

For (2), the wording is a bit unclear: Does the definition in the draft already implement necessary precautions from the BCP (I think it doesn't) or should the reader look up and follow those?

For (2) (if applicable), (3) and (4), references to specific sections in the BCP should be added.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in the OpenID4VP draft at the four passages referencing I-D.ietf-oauth-security-topics. Compare their wording with the OAuth Security BCP, clarify whether the draft incorporates or merely recommends its precautions, and add specific section references where requested. Done means the layering and guidance are unambiguous in all four passages.

Written by the indexing model from the issue text.

Assessment

Domain
documentation, security
Issue type
Documentation
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.