simplesamlphp / simplesamlphp/simplesamlphp-module-oidc

are_multiple_claim_values_allowed controls all claims in a scope

Ouverte
#360 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
PHP
Étoiles
50
Forks
28
Merge moyen
1 min
PR mergées (30 j)
2

Description

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.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par retracer la manière dont le paramètre are_multiple_claim_values_allowed est appliqué aux claims au sein d’un scope, en vous concentrant sur eduperson_affiliation et eduperson_principal_name. Comparez le comportement actuel avec les alternatives proposées au niveau des claims et au niveau du scope. Le travail est considéré comme terminé lorsque le projet dispose d’une règle convenue et prévisible pour les claims à valeur unique et à valeurs multiples, couverte par les tests pertinents.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
php
Domaine
authentication
Type d'issue
Bug
Difficulté
5/5
Temps estimé
Plus d'une semaine
Activité
Active
Clarté
À clarifier
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.