Consent-JWT / Consent-Id / PSD2-CERT header lookups are case-sensitive, so they silently fail over HTTP/2
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 78/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- scala
- 領域
- api, authentication, backend
調査の方向性
obp-api/src/main/scala/code/api/util/APIUtil.scala から始め、getConsentJWT、getConsentIdRequestHeaderValue、getPSD2-CERT、および近くにある header の検索処理を読みます。ファイル内で filter(_.name == RequestHeader...) を検索し、getRequestHeader() と動作を比較します。HTTP/2 の小文字の header 名が、大文字と小文字が混在する HTTP/1.1 リクエストと同じ consent 値を解決すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Title: Consent-JWT / Consent-Id / PSD2-CERT header lookups are case-sensitive, so they silently fail over HTTP/2
Several request-header lookups in APIUtil.scala compare the header's
name field with == against a fixed-case literal, instead of doing a
case-insensitive comparison:
// obp-api/src/main/scala/code/api/util/APIUtil.scala
def getConsentJWT(requestHeaders: List[HTTPParam]): Option[String] = {
requestHeaders.toSet.filter(_.name == RequestHeader.`Consent-JWT`).toList match {
case x :: Nil => Some(x.values.mkString(", "))
case _ => requestHeaders.toSet.filter(_.name == RequestHeader.`Consent-Id`).toList match {
case x :: Nil => Some(x.values.mkString(", "))
case _ => None
}
}
}
The same pattern appears for Consent-ID (a second, differently-cased
variant of the same header exists a few lines below Consent-Id),
PSD2-CERT, and TPP-Signature-Certificate.
Why this breaks under HTTP/2: RFC 7540 §8.1.2 requires HTTP/2 header
field names to be sent in lowercase on the wire. Any HTTP/2 client — which
is most modern HTTP clients by default when talking to a server that
advertises h2 via ALPN — sends consent-jwt, not Consent-JWT. The
== comparison against the mixed-case literal never matches, so
hasConsentJWT/getConsentJWT return false/None even though the
header is present with the correct name and value. The request then falls
through to whatever the next auth branch is, and — in our case — ends up
with a generic OBP-20001: User not logged in instead of ever reaching
Consent.checkConsent/applyConsentRulesCommon, which made this
confusing to diagnose: none of the consent-specific debug logging in
ConsentUtil.scala ever fired, because that code path was never entered.
Reproduction, against a real instance with consents.allowed=true, a
consent already created via POST /obp/v5.1.0/my/consents/IMPLICIT and
successfully answered via POST /obp/v3.1.0/banks/BANK_ID/consents/CONSENT_ID/challenge
(consent status ACCEPTED, verified independently):
# Over HTTP/2 (curl's and most clients' default for an ALPN-h2 server)
curl "https://<host>/obp/v5.1.0/users/current/user_id" \
-H "Consent-JWT: <the accepted consent's jwt>" \
-H "Consumer-Key: <consumer key>"
# -> HTTP 401 {"code":401,"message":"OBP-20001: User not logged in. Authentication is required!"}
# Identical request, forced to HTTP/1.1
curl --http1.1 "https://<host>/obp/v5.1.0/users/current/user_id" \
-H "Consent-JWT: <the same jwt>" \
-H "Consumer-Key: <consumer key>"
# -> HTTP 200 {"user_id":"..."}
Same consent, same JWT, same headers — the only difference is the wire
casing HTTP/1.1 vs HTTP/2 puts on the header name, and that alone flips
the result between "not logged in" and a correctly resolved user.
Suggested fix: compare header names case-insensitively, the way
getRequestHeader() a few lines below already does it correctly
(_.name.toLowerCase == name.toLowerCase). Applies to at least
getConsentJWT, getConsentIdRequestHeaderValue, `getPSD2-CERT`,
and the Consent-ID variant — worth a broader grep across APIUtil.scala
for the same filter(_.name == RequestHeader...) pattern to catch any
others.
Environment: reproduced against openbankproject/obp-api:latest,
commit b5a5765a185d173392c9d9c8fe36a4381097d954 (current develop HEAD
as of 2026-09-16), Kubernetes (k3s), behind an ingress-nginx TLS
termination that negotiates HTTP/2 with clients — but the bug is in the
header comparison itself and is not specific to this deployment; any
HTTP/2 client hitting these endpoints on any OBP-API instance should
reproduce it the same way.
- 主要言語
- Scala
- スター
- 1.7k
- フォーク
- 482
- 平均マージ
- 1日 12時間
- マージ済み PR(30日)
- 15
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
OpenBankProject/OBP-API のほかの issue
-
難易度 4/5 3〜5日 初心者へのやさしさ 38/100
OpenBankProject/OBP-API#2888 · コメント 1 件 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 15/100
OpenBankProject/OBP-API#2741 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
OpenBankProject/OBP-API#2624 · コメント 1 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
OpenBankProject/OBP-API#2597 ·
-
Validation email オープン
難易度 3/5 1〜2日 初心者へのやさしさ 20/100
OpenBankProject/OBP-API#2488 ·
OpenBankProject/OBP-API の issue をすべて見る
似ている issue
-
module: unknown type: bug/reported
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
OpenXiangShan/XiangShan#6623 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 70/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
ergoplatform/ergodocs#614 ·
-
area:ci enhancement requires-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
apache/datafusion-comet#6078 ·
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100