Bug 50747 - Syntax emailAddressValidDomain does not check for valid domains
Syntax emailAddressValidDomain does not check for valid domains
Status: NEEDMOREINFO
Product: UCS
Classification: Unclassified
Component: UDM (Generic)
UCS 4.4
Other Linux
: P5 normal (vote)
: ---
Assigned To: UMC maintainers
UMC maintainers
:
Depends on:
Blocks: 50748
  Show dependency treegraph
 
Reported: 2020-01-29 10:58 CET by Sönke Schwardt-Krummrich
Modified: 2020-05-20 17:18 CEST (History)
3 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 3: Simply Wrong: The implementation doesn't match the docu
Who will be affected by this bug?: 4: Will affect most installed domains
How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.137
Enterprise Customer affected?: Yes
School Customer affected?: Yes
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Regression
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 2020-01-29 10:58:35 CET
During a UCS training I noticed that the syntax emailAddressValidDomain which is used for mailPrimaryAddress does not check specified mail addresses against the local mail domain objects. Therefore the user/admin is able to specify arbitrary mail addresses.

I noticed this broken behaviour with UCS 4.4-3 and 4.4-0. 

In 4.3-3 ist seems to be ok, so I suspect a regression during the 4.3→4.4 step.
Comment 1 Toni Röhmeyer univentionstaff 2020-02-04 15:23:36 CET
I cannot reproduce the bug you described (on UCS 4.4-3).

I tried to modify a users mailPrimaryAddress to a non-existing domain and the output seems right:
"The domain part of the primary mail address is not in list of configured mail domains: foo@bar."

The responsible source code also seems fine to me.

How did you produce this error?
What domain did you use for the users mailPrimaryAddress?
Comment 2 Dirk Ahrnke univentionstaff 2020-05-20 17:18:49 CEST
2 additional occurences with 4.4-4, one of them a freshly installed pilot.
one system is unsing OX-Appsuite, the second had OX installed previously

management-console-module-udm.log (debug-level 4)

20.05.20 17:05:13.199  ADMIN       ( INFO    ) : ucr_overwrite_properties: found variable: directory/manager/web/modules/groups/group/properties/name/syntax
20.05.20 17:05:13.199  ADMIN       ( INFO    ) : ucr_overwrite_properties: found property
20.05.20 17:05:13.199  ADMIN       ( INFO    ) : ucr_overwrite_properties: set property attribute syntax to gid_ox
20.05.20 17:05:13.200  ADMIN       ( ERROR   ) : ucr_overwrite_properties: failed to set property attribute: 'string' object is not callable

# ucr search --brief syntax
directory/manager/web/modules/groups/group/properties/name/syntax: gid_ox
directory/manager/web/modules/users/user/properties/homePostalAddress/syntax: postalAddress
directory/manager/web/widget/.*/syntax: <empty>

maybe unrelated:
root@ucs01:~# ucr get directory/manager/web/modules/groups/group/properties/name/syntax
gid_ox
root@ucs01:~# ucr unset directory/manager/web/modules/groups/group/properties/name/syntax
W: The config registry variable 'directory/manager/web/modules/groups/group/properties/name/syntax' does not exist