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

objectClass:posixGroup support #37

Closed
wants to merge 4 commits into from
Closed

Conversation

cinek810
Copy link

This patch allows OpenNebula to check if user is a member of group also in case if the group has posixGroup object Class.

@jfontan
Copy link
Contributor

jfontan commented Sep 23, 2014

It would be great if it could work with old versions witch use the full DN. Maybe adding a configuration option and checking one value or the other at run time.

@cinek810
Copy link
Author

In fact the whole commit does this, I thought I have removed previos proposition. Check : #925a124, im checking the name of the filed and if is memberUid compering this with uid in all other cases with full DN.

@jfontan
Copy link
Contributor

jfontan commented Sep 24, 2014

That's not what I understood with commit d2e50ed. I'll make some tests with the proposed code. Thanks!

@cinek810
Copy link
Author

2014-09-24 12:00 GMT+02:00 Javi Fontan notifications@github.com:

That's not what I understood with commit d2e50ed
d2e50ed.
I'll make some tests with the proposed code. Thanks!

sorry for making the confusion


Reply to this email directly or view it on GitHub
#37 (comment).

@jfontan
Copy link
Contributor

jfontan commented Oct 23, 2014

I've been testing the patches and the same functionality can be done changing the configuration file:

# field that holds the user name, if not set 'cn' will be used
:user_field: 'uid'

# field name for group membership, by default it is 'member'
:group_field: 'memberuid'

# user field that that is in in the group group_field, if not set 'dn' will be used
:user_group_field: 'uid'

Maybe we can add this information to the docs but I prefer to leave the code as is.

Thanks!

@jfontan jfontan closed this Oct 23, 2014
@cinek810
Copy link
Author

but why? It's difficult to understand this for me..

@cinek810
Copy link
Author

ok, I haven't read you information about change in configuration before. Thanks!

rsmontero pushed a commit that referenced this pull request Jun 24, 2020
Signed-off-by: Frederick Borges <fborges@opennebula.io>
rsmontero pushed a commit that referenced this pull request Jun 24, 2020
Signed-off-by: Frederick Borges <fborges@opennebula.io>
rsmontero pushed a commit that referenced this pull request Oct 6, 2020
Signed-off-by: Frederick Borges <fborges@opennebula.io>
(cherry picked from commit 8302e91)
rsmontero pushed a commit that referenced this pull request Apr 13, 2021
Signed-off-by: Frederick Borges <fborges@opennebula.io>
(cherry picked from commit 8302e91)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants