#187 Populate avatar from SAML attribute#740
Conversation
879714b to
7868409
Compare
|
Can we expect this to be merged (soon)? 🤔 |
I don't expect it to happen anymore... This PR was created 211 days ago but I don't think anyone is willing to review it. |
Sad. |
|
Sorry. I was actually starting to look into it quite a while ago… but don't have a good setup for smoke testing it. Generally the code looked fine iirc. Have to revisit. |
|
Thank you very much for the work! @blizzz can we support somehow? |
Ideally extending the integration test, shibboleth need to be made to provide this. |
|
First of all, thank you very much for the work you have done. |
|
I'm trying to revive this :) |
I (and others probably) are very grateful for your revival. I hope it gets over the finish line. |
I would appreciate some testing :) My favorite SAML provider for dev (authentik) doesn't really support exposing the avatar as a SAML attribute |
Signed-off-by: Rick Bruyninckx <xactme@gmail.com> Signed-off-by: Carl Schwan <carlschwan@kde.org>
I only have a production instance so I don't know if I can test. How do I go about installing a test version of an app? |
This PR addresses #187.
The code is coppied from Lumrenion but his PR seems stale so I'm giving it a shot.
In admin configuration, a new configuration field avatar_mapping is added.
If avatar_mapping configuration field is not empty, the user wont be able to upload an image in the nextcloud profile settings
The avatar_mapping field holds a publicly available URL to the profile image of the user. As usual, the image will be downloaded to nextcloud when the user logs in via user_saml.