-
Notifications
You must be signed in to change notification settings - Fork 123
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add dynamic requested attributes #332
base: master
Are you sure you want to change the base?
Conversation
This PR has several problems:
So for each attribute we need to know not only the name but also whether it is required as well. We could require the operator to provide a list of attributes, in the format of the existing requested attributes, that will be if they were requested from the frontend. This behaviour will be useful in the case of eIDAS as we can't always request all the attributes with one call as sometimes we might want the natural person attributes and other times the legal person attributes.
|
b3a43da
to
95d6a56
Compare
I have worked the aforementioned problems, here are a couple of notes on the changes:
This is needed because eidas requires that we use the correct value for required (i.e. we can't have
|
This MR introduces the feature of populating the requested attributes of the SAML backend's request based on the attributes that were requested from the frontend.
All Submissions: