6.0.0-git
2019-03-19

[#5937] smbldap driver: don't update commented attributes
Summary smbldap driver: don't update commented attributes
Queue Passwd
Queue Version 3.0.1
Type Enhancement
State Resolved
Priority 1. Low
Owners chuck (at) horde (dot) org
Requester fabio.pedretti (at) ing (dot) unibs (dot) it
Created 2007-11-29 (4128 days ago)
Due
Updated 2007-11-30 (4127 days ago)
Assigned
Resolved 2007-11-30 (4127 days ago)
Milestone
Patch No

History
2007-11-30 21:50:25 Chuck Hagenbuch Comment #5
Assigned to Chuck Hagenbuch
State ⇒ Resolved
Reply to this comment
Committed, thanks!
2007-11-30 09:52:47 fabio (dot) pedretti (at) ing (dot) unibs (dot) it Comment #4
New Attachment: backends.php.dist.patch Download
Reply to this comment
Also attached a documentation patch for backends.php.dist .
2007-11-29 17:16:13 Chuck Hagenbuch Comment #3
State ⇒ Feedback
Reply to this comment
This looks good to me - I'll commit it in a bit if no one has any 
objections/sees any problems with it.
2007-11-29 15:13:31 fabio (dot) pedretti (at) ing (dot) unibs (dot) it Comment #2
New Attachment: smbldap.php.patch Download
Reply to this comment
A patch form smbldap.php is attached.

It also fixes a possible minor problem, where if pw_expire_time was 
not set in backends.php, it was set to current time + 2147483647 * 
86400. However seems [1] that 2147483647 is the maximux NT time 
already in seconds (would be nice if someone with a better 
understanding of samba/windows could confirm).



[1] http://lists.samba.org/archive/samba/2004-January/078175.html
2007-11-29 12:43:33 fabio (dot) pedretti (at) ing (dot) unibs (dot) it Comment #1
Type ⇒ Enhancement
State ⇒ New
Priority ⇒ 1. Low
Summary ⇒ smbldap driver: don't update commented attributes
Queue ⇒ Passwd
Reply to this comment
In my LDAP tree I have only sambaLMPassword and sambaNTPassword, but 
not sambaPwdLastSet and sambaPwdMustChange (I need these password 
hashes for FreeRADIUS).



If an attribute in config/backends.php is commented would be nice to 
not set it in the LDAP. So I could comment:

#        'pw_set_attribute' => 'sambaPwdLastSet',

#        'pw_expire_attribute' => 'sambaPwdMustChange',

#        'pw_expire_time' => 180, // number of days until samba 
password expires

and these won't be updated.



Ideally, smbldap and ldap should be the same driver, where one can 
enable only the needed attributes.

Saved Queries