nodeSolidServer / nodeSolidServer/node-solid-server

Rethink authentication configuration

オープン
#672 コメント 11 件 リアクション 3 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

authentication in progress semver-major
主要言語
JavaScript
スター
1.8k
フォーク
308
PR マージ指標
30日以内にマージされた PR はありません

説明

Right now, the server supports two different auth modes, confusingly named auth: "tls" and auth: "oidc", even though that doesn't cover exactly what they do. The main problem is that this one setting gives the impression that both methods are mutually exclusive, whereas they are not.

Discussions such as #667, #671, https://github.com/solid/solid-auth-client/issues/44, https://github.com/solid/solid/issues/138, https://github.com/solid/oidc-auth-manager/issues/20 show that there is a need for more clarity, and a better configuration option.

This issue tracks general progress and will act as a tracker for multiple sub-issues.
Here are the steps we need to take in my opinion:

  • Have one single contract for the entire node-solid-server to track a) the WebID of the logged-in agent b) the method through which that agents has logged in
  • Document the above contract
  • Ensure that “ask for a client certificate on every resource (WebID-TLS)” can be enabled and disabled independently of other features
  • Ensure that “act as an OIDC identity provider” can be enabled and disabled independently of other features
  • Ensure that “log in with OIDC through username/password” can be enabled and disabled independently of other features
  • Ensure that “log in with OIDC through client certificates (WebID-TLS)” can be enabled and disabled independently of other features
  • Ensure that “accept bearer tokens” can be enabled and disabled independently of other features
  • Examine whether we want to always translate successful logins into an Express cookie-based session
  • Examine whether we want to allow turning on or off cookie-based sessions
  • Create a configuration section that allows to enable and disable all of the above options independently
  • Update the init generator to create the right config
  • Remove the old auth config section

Feedback welcome.

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

まず、未解決の設計を理解するために、ディスカッション #667、#671、およびリンクされている認証関連の issue を読んでください。次に、issue で言及されている既存の auth 設定と init generator を追跡してください。完了条件は、古い auth セクションを、一覧にあるログイン、トークン、証明書、identity provider、cookie-session の各オプションに対する、文書化された独立した制御に置き換えることです。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
express, javascript, node.js
領域
authentication, backend
issue の種類
機能追加
難易度
5/5
見積もり時間
1週間以上
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
20/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。