Skip to content
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

new group processor based on attribute #24

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

howkymike
Copy link
Contributor

Default attribute name: alfrescoGroupName
To make it work I deleted groupProperties.put(ContentModel.PROP_AUTHORITY_NAME, groupName); from KeycloakUserRegistry.java

@AFaust AFaust added change for interested parties Others may want to apply this change in their fork of this project wontfix This will not be worked on labels Oct 17, 2021
@AFaust
Copy link
Member

AFaust commented Oct 17, 2021

Note: The "wontfix" label applies to the specific way the change in this PR is done, not to the idea behind it. As outlined in #18 I plan to work on a flexible attribute mapping scheme (for users and groups), and will likely combine this with a configuration flag that would allow those mappings to specify the Alfresco authority name. For the sake of guarding against accidental duplicates, the default will remain that the ID is used for the name, but admins will then have a choice to change that behaviour.

Thanks @howkymike for providing the change so anyone interested in this have a starting point to use this until I have made the above available.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
change for interested parties Others may want to apply this change in their fork of this project wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants