-
Notifications
You must be signed in to change notification settings - Fork 233
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
ipauser: Support for External IdP attributes. #1055
Conversation
fc622aa
to
a8a9700
Compare
a8a9700
to
425b8a3
Compare
There's still an issue with "timing" on the tests, as the command to create the IDP entry is not working when executed inside the testing container, but always succeed in a clean CentOS 9 Stream VM. |
425b8a3
to
4ca315c
Compare
It was not timing after all... The |
665630c
to
8d678e1
Compare
idp and idp_user_id parameters are not in the spec anymore. |
8d678e1
to
722ebba
Compare
Add support for 'idp' and 'idp_user_id' to ipauser plugin. FreeIPA 4.10.0 is required for both attributes.
722ebba
to
6fa8223
Compare
@t-woerner PR is ready for review again. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Existing downstream tests have been successfully passed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Add support for 'idp' and 'idp_user_id' to ipauser plugin.
FreeIPA 4.10.0 is required for both attributes.