Added attribute option that allow to specify users-groups assoc #15

Merged
merged 1 commit into from Oct 6, 2011

Conversation

Projects
None yet
3 participants

goetas commented Sep 29, 2011

No description provided.

Owner

thorin commented Sep 29, 2011

Thank you for the customization.
I'll have to do a couple of changes on migrate.rb, and documentation first.

Which attribute are you using instead of :dn?

goetas commented Sep 29, 2011

ok.
I'm using "uid" as identification attribute.

In my LDAP database, uid represents a username.

@biow0lf biow0lf commented on the diff Oct 5, 2011

db/migrate/201108021245_change_settings_name.rb
@@ -15,6 +15,7 @@ class ChangeSettingsName < ActiveRecord::Migration
settings[:sync_user_attributes] = false
settings[:attr_member] = 'member'
settings[:class_group] = 'group'
+ settings[:attr_user_memberid] = 'dn'
@biow0lf

biow0lf Oct 5, 2011

Changing old migration is bad style. Howto current users will migrate?

@goetas

goetas Oct 5, 2011

the old users will continue to use 'dn' as default attr_user_memberid field.
may be useful add something like this on line 111 of auth_source_ldap_patch.rb:

       attr_user_memberid = ' dn'
else
        attr_user_memberid = settings[:attr_user_memberid]

??

@thorin

thorin Oct 5, 2011

Owner

The best is to create a new migrate file (20111005_new_add_user_memberid_setting.rb for example).

@goetas

goetas Oct 5, 2011

i'm new to ruby, and i think that i'm not able to do this.

thorin was assigned Oct 6, 2011

@thorin thorin merged commit f7f03d1 into thorin:master Oct 6, 2011

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment