Univention Bugzilla – Full Text Bug Listing |
Summary: | Mail domain name not validated | ||
---|---|---|---|
Product: | UCS | Reporter: | Dirk Wiesenthal <wiesenthal> |
Component: | UMC - Mail | Assignee: | Alexander Kläser <klaeser> |
Status: | CLOSED FIXED | QA Contact: | Florian Best <best> |
Severity: | normal | ||
Priority: | P5 | CC: | best, gohmann, klaeser, kramer, walkenhorst |
Version: | UCS 4.0 | ||
Target Milestone: | UCS 4.0-1-errata | ||
Hardware: | Other | ||
OS: | Linux | ||
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): | Error handling, Usability | |
Max CVSS v3 score: |
Description
Dirk Wiesenthal
2014-04-15 11:19:10 CEST
The manual says: "The name of a mail domain may only be composed of lowercase letters, the figures 0-9, full stops and hyphens." This really should be validated. In my tests, I could create an arbitrary mail domain without any error occurring. r 59126 added a new syntax class to validate the mail domain name. Also adpated the de.po for the german translation debian/changelog univention-directory-manager-modules (10.0.29-34) * Bug #34552: Validate the mail domain name Successful build Package: univention-directory-manager-modules Version: 10.0.29-34.1298.201503171542 Branch: ucs_4.0-0 Scope: errata4.0-1 r 59127 yaml file If you want to restore the old behavior/syntax class you can overwrite the syntax from the cli: ucr set directory/manager/web/modules/mail/domain/properties/name/syntax='string' Also added this information to the yaml file: r 59130 As requested I changed the error messages and fixed the regex statement. r 59134 debian/changelog univention-directory-manager-modules (10.0.29-35) * Bug #34552: Adapted the regex and the messages Successful build Package: univention-directory-manager-modules Version: 10.0.29-35.1299.201503171648 Branch: ucs_4.0-0 Scope: errata4.0-1 Host: dimma.knut.univention.de Logfile Path: /var/univention/buildsystem2/logs/ucs_4.0-0-0-errata4.0-1/ucs_4.0-0-errata4.0-1.univention-directory-manager-modules.201503171648.log.bz2 r 59136 yaml file adapted the fix number REOPEN: code change The reg exp within the syntax matches also to the empty string which is not desired. Besides that, there is already an existing syntax class that IMHO fits better: "DNS_Name" Currently I cannot see any reason why this syntax class should not be reused. (In reply to Sönke Schwardt-Krummrich from comment #5) > REOPEN: code change > > The reg exp within the syntax matches also to the empty string which is not > desired. Besides that, there is already an existing syntax class that IMHO > fits better: "DNS_Name" > Currently I cannot see any reason why this syntax class should not be reused. Agreed. I take this bug such that it can be published tomorrow. univention-directory-manager-modules (10.0.29-36) [r59362]: * Bug #34552: replace mail_domain_name syntax with DNS_Name OK: fix [en|ger] OK: YAML |