nodeSolidServer / nodeSolidServer/node-solid-server

issuer URIs with a path part don't work

未關閉
#1,265 2 則留言 0 個 reaction 已指派 0 人 在 GitHub 檢視

還沒有人認領這個 Issue。

bug
主要語言
JavaScript
星號
1.8k
分支
308
PR 合併指標
30 天內沒有已合併 PR

描述

i originally opened this as https://github.com/solid/oidc-auth-manager/issues/44 but perhaps i should have started here to target the right audience.

oidc-auth-mananger seems to make an invalid assumption that all OIDC Issuers are URIs with an empty path part. if a webid has an OIDC Issuer URI with a path part (like my webid https://zenomt.zenomt.com/card.ttl#me with issuer https://zenomt.com/oidc/), it can never be matched against the id_token iss claim because the discovered issuer URI is reduced to its origin before being compared. see these lines in preferred-provider.js.

OIDC Issuers are allowed to have path parts.

貢獻指南

開啟貢獻指南

從這裡開始

  1. 先讀完整個 Issue,再讀專案的貢獻指南。
  2. 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
  3. Fork 儲存庫,在一個分支上完成修改。
  4. 送出 Pull Request,並在描述裡引用這個 Issue 編號。

研究方向

從 src/preferred-provider.js 第 70-73 行附近開始,這裡發現的 issuer URI 在與 id_token 的 iss claim 比較之前會被縮短。驗證 OpenID Connect 對 issuer 路徑部分的要求,然後在比對時保留路徑。當 https://zenomt.com/oidc/ 這樣的 issuer 能夠與對應的 iss claim 相符時,即表示完成。

由索引模型根據 Issue 內容生成。

評估

技術堆疊
javascript
領域
authentication
Issue 類型
缺陷
難度
2/5
預估耗時
1-3 小時
活躍度
停滯
描述清晰度
描述清楚
新手友好度
55/100

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。