simplesamlphp / simplesamlphp/simplesamlphp-module-oidc

are_multiple_claim_values_allowed controls all claims in a scope

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

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

主要言語
PHP
スター
50
フォーク
28
平均マージ
1分
マージ済み PR(30日)
2

説明

If I set are_multiple_claim_values_allowed to true in a custom scope eduperson, which I need for eduperson_affiliation, eduperson_principal_name also becomes a list even if it contains only one value (and is defined as single-valued).
Maybe the are_multiple_claim_values_allowed setting should live at the claim definition level instead of scope?

Or: Always send single valued claims as string and expand to list if are_multiple_claim_values_allowed is set to true at the scope level, this would be the least intrusive refactor. But the downside is that this behaviour would be unpredictable for the RP.

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

このリポジトリのコントリビューションガイドは索引されていません

はじめの一歩

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

調査の方向性

まず、スコープ内の claims に設定 are_multiple_claim_values_allowed がどのように適用されるかを追跡し、eduperson_affiliationeduperson_principal_name に焦点を当てます。現在の動作を、提案されている claim レベルおよびスコープレベルの代替案と比較します。プロジェクトで単一値 claims と複数値 claims に対する合意済みの予測可能なルールが定められ、関連するテストでカバーされていれば完了です。

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

評価

技術スタック
php
領域
authentication
issue の種類
バグ
難易度
5/5
見積もり時間
1週間以上
活発さ
活発
明瞭さ
説明が足りない
初心者へのやさしさ
35/100

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

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