simplesamlphp / simplesamlphp/simplesamlphp-module-oidc
are_multiple_claim_values_allowed controls all claims in a scope
Nessuno ha ancora preso questa issue.
- Lingua principale
- PHP
- Stelle
- 50
- Fork
- 28
- Merge medio
- 1m
- PR unite (30g)
- 2
Descrizione
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.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia tracciando il modo in cui l'impostazione are_multiple_claim_values_allowed viene applicata ai claims all'interno di uno scope, concentrandoti su eduperson_affiliation e eduperson_principal_name. Confronta il comportamento attuale con le alternative proposte a livello di claim e a livello di scope. Il lavoro è considerato completato quando il progetto dispone di una regola concordata e prevedibile per i claim a valore singolo e a valori multipli, coperta dai test pertinenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- php
- Ambito
- authentication
- Tipo di issue
- Bug
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Attiva
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 35/100