[opensuse-buildservice] [PATCH] Add changing password feature in the UI
Hi, Please help to review these 2 patches. They're adding the feature of changing passwords in the UI(attached the screenshot) and works properly with common authentication mode and LDAP mode. This new feature is configurable and can be easily disabled by the user who does not want it. Thanks vivian
Hello Vivian, On Friday 06 August 2010 07:32:24 Zhang, Vivian wrote:
Hi,
Please help to review these 2 patches. They're adding the feature of changing passwords in the UI(attached the screenshot) and works properly with common authentication mode and LDAP mode. This new feature is configurable and can be easily disabled by the user who does not want it.
They look good to me. so I have imported them into git master tree. I just wonder, if we really want to have an option to dis-allow the password change of the users. Is there a real use case, where this should not be allowed ? People who want to test this feature can use current packages from openSUSE:Tools:Unstable. Thank you and welcome new OBS hacker :) bye adrian -- Adrian Schroeter SUSE Linux Products GmbH email: adrian@suse.de -- To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-buildservice+help@opensuse.org
On Fri, Aug 06, Adrian Schröter wrote:
Please help to review these 2 patches. They're adding the feature of changing passwords in the UI(attached the screenshot) and works properly with common authentication mode and LDAP mode. This new feature is configurable and can be easily disabled by the user who does not want it.
They look good to me. so I have imported them into git master tree.
I just wonder, if we really want to have an option to dis-allow the password change of the users. Is there a real use case, where this should not be allowed ?
In case of LDAP, this use-case might become valid as a lot of LDAP based infrastructures are using AD or any other directory where changing password must be done elsewhere. -- With best regards, Carsten Hoeger
On 08/06/2010 08:52 AM, Adrian Schröter wrote:
Hello Vivian,
On Friday 06 August 2010 07:32:24 Zhang, Vivian wrote:
Hi,
Please help to review these 2 patches. They're adding the feature of changing passwords in the UI(attached the screenshot) and works properly with common authentication mode and LDAP mode. This new feature is configurable and can be easily disabled by the user who does not want it.
They look good to me. so I have imported them into git master tree.
I just wonder, if we really want to have an option to dis-allow the password change of the users. Is there a real use case, where this should not be allowed ?
Maybe for iChain based installations? ;-) Greetings -- Thomas Schmidt (tschmidt [at] suse.de) SUSE Linux Products GmbH :: Research & Development :: Tools "memory management can be a bitch" (linux/kernel/fork.c:11) -- To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-buildservice+help@opensuse.org
On Friday 06 August 2010 11:28:55 Thomas Schmidt wrote:
On 08/06/2010 08:52 AM, Adrian Schröter wrote:
Hello Vivian,
On Friday 06 August 2010 07:32:24 Zhang, Vivian wrote:
Hi,
Please help to review these 2 patches. They're adding the feature of changing passwords in the UI(attached the screenshot) and works properly with common authentication mode and LDAP mode. This new feature is configurable and can be easily disabled by the user who does not want it.
They look good to me. so I have imported them into git master tree.
I just wonder, if we really want to have an option to dis-allow the password change of the users. Is there a real use case, where this should not be allowed ?
Maybe for iChain based installations? ;-)
Sure, but we can check for iChain mode in that case. Not a reason to add another option IMHO. bye adrian -- Adrian Schroeter SUSE Linux Products GmbH email: adrian@suse.de -- To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-buildservice+help@opensuse.org
participants (4)
-
Adrian Schröter
-
Carsten Hoeger
-
Thomas Schmidt
-
Zhang, Vivian