Bug 39882 - Broken validation for MultiInput fields - DNS SRV record for example
Broken validation for MultiInput fields - DNS SRV record for example
Status: REOPENED
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 5.0
Other Linux
: P5 normal with 2 votes (vote)
: ---
Assigned To: UMC maintainers
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2015-11-11 16:22 CET by Philipp Hahn
Modified: 2021-04-27 10:45 CEST (History)
3 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 1: Cosmetic issue or missing function but workaround exists
Who will be affected by this bug?: 3: Will affect average number of installed domains
How will those affected feel about the bug?: 1: Nuisance – not a big deal but noticeable
User Pain: 0.017
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): External feedback, Usability
Max CVSS v3 score:


Attachments
Screenshot UMC DNS Module for SRV (62.99 KB, image/png)
2019-02-27 20:09 CET, Philipp Hahn
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Philipp Hahn univentionstaff 2015-11-11 16:22:34 CET
When adding a now DNS SRV RR, the 3rd fields "Extension" is marked with an asterisk as required; it even turns red when not entered.
But the record can still be created when saving.

Extension is some MS extension used fro "ForestDnsZones" and "Default-First-Site-Name._sites.ForestDnsZones", but not required otherwise.
Comment 2 Florian Best univentionstaff 2016-02-19 14:00:07 CET
This is not easy to fix as the widget is only a subsyntax of a MultiInput which inherits the 'required' flag from the parent property description. There is no way to defined the 'required' flag at the syntax class.
Comment 3 Stefan Gohmann univentionstaff 2019-01-03 07:23:58 CET
This issue has been filled against UCS 4.1. The maintenance with bug and security fixes for UCS 4.1 has ended on 5st of April 2018.

Customers still on UCS 4.1 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.
Comment 4 Philipp Hahn univentionstaff 2019-02-27 19:16:07 CET
Problem persists with UCS-4.4
Comment 5 Philipp Hahn univentionstaff 2019-02-27 20:09:09 CET
Created attachment 9871 [details]
Screenshot UMC DNS Module for SRV

There also seems to be a problem that an error in *one* sub-field is propagated to an error message for *all* fields:

> class dnsSRVLocation(complex):
>   subsyntaxes = [(_('Priority'), integer), (_('Weighting'), integer), (_('Port'), integer), (_('Server'), dnsHostname)]

"Server" clearly is not of type "integer", but nevertheless flagged as containing an error with "Value must be a number!".
Comment 6 Philipp Hahn univentionstaff 2021-04-27 10:45:50 CEST
Still with UCS 5.0-0