Bug 37532 - User Password Admins can't change passwords via UMC
User Password Admins can't change passwords via UMC
Status: RESOLVED WONTFIX
Product: UCS
Classification: Unclassified
Component: UMC - Domain management (Generic)
UCS 4.0
Other Linux
: P5 normal (vote)
: ---
Assigned To: UMC maintainers
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2015-01-13 14:47 CET by Florian Best
Modified: 2019-01-03 07:17 CET (History)
2 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Usability
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Best univentionstaff 2015-01-13 14:47:31 CET
User Password Admins are regularly able to change the password of users.
After they open a user object, default values for Comboboxes are set if the user doesn't have some attributes set. This could e.g. be:
* ctx-* flags (fixed by Bug #
* extended attributes
* settings from any app, e.g. fetchmail sets 'Protocol' to 'IMAP'
* primaryGroup for users without 'posix' option

If default values are set these values are sent along with the save-request resulting in 'Access Denied' because the password-admin has only the LDAP-access rights to change the 'password' attribute of that user.
Comment 1 Stefan Gohmann univentionstaff 2015-02-13 16:14:50 CET
Is this still valid?
Comment 2 Florian Best univentionstaff 2015-02-16 13:11:21 CET
(In reply to Stefan Gohmann from comment #1)
> Is this still valid?
yes
Comment 3 Stefan Gohmann univentionstaff 2015-02-17 07:52:19 CET
(In reply to Florian Best from comment #2)
> (In reply to Stefan Gohmann from comment #1)
> > Is this still valid?
> yes

OK, but it happens only if you have installed Fetchmail or you have users without the POSIX option?
Comment 4 Florian Best univentionstaff 2015-02-17 09:53:43 CET
(In reply to Stefan Gohmann from comment #3)
> (In reply to Florian Best from comment #2)
> > (In reply to Stefan Gohmann from comment #1)
> > > Is this still valid?
> > yes
> 
> OK, but it happens only if you have installed Fetchmail or you have users
> without the POSIX option?
Or any other app/extension which have extended attributes which adds comboboxes to users/user.
Comment 5 Florian Best univentionstaff 2015-02-18 12:39:37 CET
It is caused because ComboBoxes are setting default values if no value was provided.
Comment 6 Alexander Kläser univentionstaff 2015-02-19 12:39:38 CET
I see the following options:
* Disable those fields for which LDAP write access is not granted.
* Upon save, validate which fields may be written to LDAP and prompt a confirmation dialogue that states that only the following values may be written.
* Simply write all values into LDAP that may be written and ignore the others (or display a hint "BTW, only the following values could be saved").
Comment 7 Florian Best univentionstaff 2015-02-19 12:44:02 CET
(In reply to Alexander Kläser from comment #6)
> I see the following options:
> * Disable those fields for which LDAP write access is not granted.
How to detect this? by writing a LDAP ACL parser?
> * Upon save, validate which fields may be written to LDAP and prompt a
> confirmation dialogue that states that only the following values may be
> written.
How to detect this case? Always asking for which values should be changed is disturbing.
> * Simply write all values into LDAP that may be written and ignore the
> others (or display a hint "BTW, only the following values could be saved").
Then we would have to make for every attribute one ldap-modify request which will break if 2 attributes depend on each other.

I would see also the option to only send values from widgets which are visible, everything which is invisible on purpose can use the HiddenInput widget which is already currently used.
Comment 8 Florian Best univentionstaff 2015-02-19 12:44:53 CET
(In reply to Florian Best from comment #7)
> I would see also the option to only send values from widgets which are
> visible, everything which is invisible on purpose can use the HiddenInput
> widget which is already currently used.
err, well this does not work as there can ofc. also be visible widgets :(
Comment 9 Stefan Gohmann univentionstaff 2019-01-03 07:17:01 CET
This issue has been filled against UCS 4.0. The maintenance with bug and security fixes for UCS 4.0 has ended on 31st of May 2016.

Customers still on UCS 4.0 are encouraged to update to UCS 4.3. Please contact
your partner or Univention for any questions.

If this issue still occurs in newer UCS versions, please use "Clone this bug" or simply reopen the issue. In this case please provide detailed information on how this issue is affecting you.