search

SURFnet / sp-dashboard

SURFconext Service Provider Dashboard
Apache License 2.0
5 stars 2 forks source link

Niet alle attributen worden in de ARP opgenomen #807

Closed phavekes closed 1 week ago

phavekes commented 1 week ago

This issue is imported from pivotal - Originaly created at Nov 2, 2017 by Bart Geesink

Als ik alle attributen aanvink om in de ARP te laten landen komen alleen deze attributen mee: eduPersonAffiliation eduPersonEntitlement eduPersonPrincipalName eduPersonScopedAffiliation eduPersonTargetedID preferredLanguage schacHomeOrganization uid

phavekes commented 1 week ago

Zoals besproken, alle attributen komen op mijn development-omgeving mee. Ook na copy. Graag hertest met laatste versie, anders moeten we even meekijken wat er fout gaat op test2. (Joris Steyn - Nov 3, 2017)

phavekes commented 1 week ago

Alle RequestedAttribute attributes worden correct vertaald naar ARP attributen. Zie bijgevoegde XML voor een voorbeeld. Ook alle ARP attributen worden in de JSON export en in de SAML export meegegeven. LET OP: alleen geldige ARP attributen worden meegenomen. Zie https://github.com/OpenConext/OpenConext-deploy/blob/master/roles/manage-server/files/metadata_configuration/saml20_sp.schema.json#L127 voor alle geldige attributen. (Okke Harsta - Dec 5, 2017)

phavekes commented 1 week ago

In de manage-gui kan je bijgevoegde XML importeren en valideren dat alle RequestedAttribute waardes - indien geldig - worden vertaald naar de ARP (Okke Harsta - Dec 5, 2017)

phavekes commented 1 week ago

Het probleem hier was dat wij voor de attributen welke niet doorkomen een FriendlyName meegaven welke niet overeen kwam met het laatste gedeelte van de urn. Ik weet niet of FriendlyName daar echt aan moet voldoen volgens de specs, maar we hebben dit nu aan de dashboard kant gefixed en dit werkt. (Joris Steyn - Dec 8, 2017)

phavekes commented 1 week ago

We missen er nog een: urn:mace:terena.org:attribute-def:schacHomeOrganizationType (Bart Geesink - Dec 14, 2017)

phavekes commented 1 week ago

Was een typo, fixed in https://github.com/SURFnet/sp-dashboard/pull/97 (Joris Steyn - Dec 14, 2017)

phavekes commented 1 week ago

Getest, alle attributen komen nu mee. (Bart Geesink - Dec 18, 2017)