simplesamlphp / simplesamlphp/simplesamlphp-module-oidc
are_multiple_claim_values_allowed controls all claims in a scope
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PHP
- Sterne
- 50
- Forks
- 28
- Ø Merge
- 1 Min.
- Gemergte PRs (30 T.)
- 2
Beschreibung
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.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit nachzuverfolgen, wie die Einstellung are_multiple_claim_values_allowed auf Claims innerhalb eines Scopes angewendet wird, mit besonderem Fokus auf eduperson_affiliation und eduperson_principal_name. Vergleiche das aktuelle Verhalten mit den vorgeschlagenen Alternativen auf Claim-Ebene und Scope-Ebene. Als abgeschlossen gilt die Aufgabe, wenn das Projekt eine abgestimmte, vorhersehbare Regel für einwertige und mehrwertige Claims hat, die durch relevante Tests abgedeckt ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- authentication
- Issue-Typ
- Bug
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Aktiv
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 35/100