6.0.0-beta1
7/22/25

[#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 11/29/2007 (6445 days ago)
Due
Updated 11/30/2007 (6444 days ago)
Assigned
Resolved 11/30/2007 (6444 days ago)
Milestone
Patch No

History
11/30/2007 09:50:25 PM Chuck Hagenbuch Comment #5
Assigned to Chuck Hagenbuch
State ⇒ Resolved
Reply to this comment
Committed, thanks!
11/30/2007 09:52:47 AM 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 .
11/29/2007 05:16:13 PM 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.
11/29/2007 03:13:31 PM 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
11/29/2007 12:43:33 PM fabio (dot) pedretti (at) ing (dot) unibs (dot) it Comment #1
Priority ⇒ 1. Low
Type ⇒ Enhancement
Summary ⇒ smbldap driver: don't update commented attributes
Queue ⇒ Passwd
State ⇒ New
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