Bug 39952 - Setting contact information does not allow to set additional attributes
Setting contact information does not allow to set additional attributes
Status: RESOLVED WONTFIX
Product: UCS
Classification: Unclassified
Component: Self Service
UCS 4.2
Other Linux
: P5 enhancement (vote)
: ---
Assigned To: UMC maintainers
UMC maintainers
:
: 39982 (view as bug list)
Depends on:
Blocks: 51654
  Show dependency treegraph
 
Reported: 2015-11-13 22:40 CET by Sönke Schwardt-Krummrich
Modified: 2020-07-09 11:08 CEST (History)
4 users (show)

See Also:
What kind of report is it?: Feature Request
What type of bug is this?: 2: Improvement: Would be a product improvement
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): Roadmap discussion
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sönke Schwardt-Krummrich univentionstaff 2015-11-13 22:40:02 CET
The external send method allows to use other UDM properties than for sms and email. But the UI for setting contact information does not allow to use other UDM properties.
Thus the admin is able to configure e.g. a XMPP method and use an existing UDM property but the user is not able to maintain its own XMPP address via the self service module.
Comment 1 Daniel Tröder univentionstaff 2015-11-30 10:00:20 CET
*** Bug 39982 has been marked as a duplicate of this bug. ***
Comment 2 Florian Best univentionstaff 2017-04-28 11:59:36 CEST
I think the best would be to just re-enable users/self flavor in UMC which has all required features.
Comment 3 Daniel Tröder univentionstaff 2017-04-28 12:03:39 CEST
The UI of the UMC has been judged to be unfitting for end-users.

BTW: a "Edit contact information" link would neatly fit into the side panel → "User settings".
Comment 4 Florian Best univentionstaff 2017-04-28 12:08:37 CEST
(In reply to Daniel Tröder from comment #3)
> The UI of the UMC has been judged to be unfitting for end-users.
If so, we can adjust the layout of it. We could even outsource the UDM's flavor into an own page if necessary.
The benefit would be that all of the required functionality is already configurable via extended attributes.
Comment 5 Florian Best univentionstaff 2017-04-28 12:09:11 CEST
→ No need for UCR variables, no need for additional Code.
Comment 6 Daniel Tröder univentionstaff 2017-04-28 12:12:53 CEST
Sounds great!
Comment 7 Daniel Tröder univentionstaff 2017-04-28 12:20:06 CEST
(In reply to Daniel Tröder from comment #6)
> Sounds great!
Actually not. The idea is better suited for Bug #44500.

The intention of this bug is to be able to edit the contact data for the activated send-method (eg XMPP) in the password reset module, not in some other place. This makes for a better UX.
Comment 8 Florian Best univentionstaff 2017-04-28 13:12:07 CEST
It works e.g. with the following already:

# ucr unset umc/module/udm/users/self/disabled
# eval "$(ucr shell)"
# udm settings/extended_attribute modify --dn "cn=UniventionPasswordSelfServiceEmail,cn=custom attributes,cn=univention,$ldap_base" --append module=users/self
# udm settings/extended_attribute modify --dn "cn=UniventionPasswordSelfServiceMobile,cn=custom attributes,cn=univention,$ldap_base" --append module=users/self
# udm policies/umc modify --dn cn=default-umc-users,cn=UMC,cn=policies,$ldap_base --append allow="cn=udm-self,cn=operations,cn=UMC,cn=univention,$ldap_base"

# firefox http://$(hostname -f)/univention/management/?module=udm&flavor=users/self

→ works so far, if one sets up the correct LDAP ACL's, e.g.:

"access to attrs=univentionPasswordSelfServiceMobile,univentionPasswordSelfServiceEmail by self write by * +0 break"
Comment 9 Stefanie Schneider univentionstaff 2018-02-26 13:05:43 CET
(In reply to Florian Best from comment #8)
> "access to
> attrs=univentionPasswordSelfServiceMobile,univentionPasswordSelfServiceEmail
> by self write by * +0 break"


it only works if the user is allowed to create the objectClass UniventionPassword SelfService:

access to attrs=objectClass,univentionPasswordSelfServiceMobile,univentionPasswordSelfServiceEmail
   by self write
   by * +0 break

Restricting the access to a specific ObjectClass unfortunately does not work. So this will give a user the ability to create ObjectClasses
Comment 10 Florian Best univentionstaff 2018-03-20 11:08:34 CET
(In reply to Stefanie Schneider from comment #9)
> (In reply to Florian Best from comment #8)
> > "access to
> > attrs=univentionPasswordSelfServiceMobile,univentionPasswordSelfServiceEmail
> > by self write by * +0 break"
> 
> 
> it only works if the user is allowed to create the objectClass
> UniventionPassword SelfService:
> 
> access to
> attrs=objectClass,univentionPasswordSelfServiceMobile,
> univentionPasswordSelfServiceEmail
>    by self write
>    by * +0 break
> 
> Restricting the access to a specific ObjectClass unfortunately does not
> work. So this will give a user the ability to create ObjectClasses

This would be a massive security hole especially in UCS@school environments.
There is a way to restrict ACL's to a specifc object class:

access to objectClass value=UniventionPasswordSelfService
    by self write
    by * +0 break

I think this should work, but I did not test it.
Comment 11 Stefanie Schneider univentionstaff 2018-04-04 11:01:31 CEST
Restriction to certain Objectclass works. See final Cool Solution:
https://wiki.univention.de/index.php/Cool_Solution_-_User_Self-Service_with_extended_Attributes
Comment 14 Ingo Steuwer univentionstaff 2020-07-03 20:56:03 CEST
This issue has been filed against UCS 4.2.

UCS 4.2 is out of maintenance and many UCS components have changed in later releases. Thus, this issue is now being closed.

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