Bug 53794 - dns/reverse_zone: ValueError
dns/reverse_zone: ValueError
Status: NEW
Product: UCS
Classification: Unclassified
Component: UMC - DNS
UCS 5.0
Other Linux
: P5 normal (vote)
: ---
Assigned To: UMC maintainers
UMC maintainers
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2021-09-15 22:09 CEST by Florian Best
Modified: 2021-09-15 22:11 CEST (History)
0 users

See Also:
What kind of report is it?: Development Internal
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):
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 2021-09-15 22:09:01 CEST
If --set contact= does not contain a "@" character:

udm dns/reverse_zone create --set subnet=13.213.187 --set 'zonettl=9 JO' --set contact=rul0tvfhqm --set serial=21704 --set 'refresh=6 BR' --set 'retry=6
GY' --set 'expire=7 TD' --set 'ttl=4 BF' --append nameserver=zgkmt.djtfl --append nameserver=bsyhj.vljll --append nameserver=azywp.lfpxv

Traceback (most recent call last):
  File "/usr/share/univention-directory-manager-tools/univention-cli-server", line 213, in doit
    output = univention.admincli.admin.doit(arglist)
  File "/usr/lib/python3/dist-packages/univention/admincli/admin.py", line 380, in doit
    out = _doit(arglist)
  File "/usr/lib/python3/dist-packages/univention/admincli/admin.py", line 650, in _doit
    out.extend(cli.create(input, append, ignore_exists, parsed_options, parsed_append_options, parsed_remove_options, policy_reference))                                                                                                      
  File "/usr/lib/python3/dist-packages/univention/admincli/admin.py", line 678, in create
    return self._create(self.module_name, self.module, self.dn, self.lo, self.position, self.superordinate, *args, **kwargs)                                                                                                                  
  File "/usr/lib/python3/dist-packages/univention/admincli/admin.py", line 731, in _create
    dn = object.create()
  File "/usr/lib/python3/dist-packages/univention/admin/handlers/__init__.py", line 543, in create
    dn = self._create(response=response, serverctrls=serverctrls)
  File "/usr/lib/python3/dist-packages/univention/admin/handlers/__init__.py", line 1255, in _create
    al.extend(self._ldap_modlist())
  File "/usr/lib/python3/dist-packages/univention/admin/handlers/dns/reverse_zone.py", line 237, in _ldap_modlist
    soa = b'%s %s %s %s %s %s %s' % (self['nameserver'][0].encode('UTF-8'), escapeSOAemail(self['contact']).encode('UTF-8'), self['serial'].encode('UTF-8'), refresh, retry, expire, ttl)                                                     
  File "/usr/lib/python3/dist-packages/univention/admin/handlers/dns/__init__.py", line 82, in escapeSOAemail
    raise ValueError()
ValueError
Comment 1 Florian Best univentionstaff 2021-09-15 22:11:01 CEST
 81 »   'contact': univention.admin.property(
 82 »   »   short_description=_('Contact person'),
 83 »   »   long_description=_('The email address of the person responsible for this zone.'),
 84 »   »   syntax=univention.admin.syntax.string,

→ We should change the syntax class to emailAddress!