Bug 53829 – Exception when giving seconds in accountActivationDate

Bug 53829 - Exception when giving seconds in accountActivationDate


Summary:	Exception when giving seconds in accountActivationDate

Status:	NEW

Product:	UCS
Classification:	Unclassified
Component:	UDM (Generic)
Version:	UCS 5.0
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	---
Assigned To:	UMC maintainers
QA Contact:	UMC maintainers

URL:
Keywords:

Depends on:	53631
Blocks:
	Show dependency tree / graph

Reported:	2021-09-23 18:19 CEST by Florian Best
Modified:	2022-11-23 18:41 CET (History)
CC List:	5 users (show)

See Also:	55452
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
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Florian Best

2021-09-23 18:19:50 CEST

udm users/user create --set 'accountActivationDate=2039-12-07 12:12:23 Europe/Tirane'

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 649, 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 677, 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 730, in _create
    dn = object.create()
  File "/usr/lib/python3/dist-packages/univention/admin/handlers/__init__.py", line 540, in create
    self._ldap_pre_ready()
  File "/usr/lib/python3/dist-packages/univention/admin/handlers/users/user.py", line 1695, in _ldap_pre_ready
    if self.hasChanged('disabled') and self['disabled'] == '0' and not self.hasChanged('accountActivationDate'):
  File "/usr/lib/python3/dist-packages/univention/admin/handlers/users/user.py", line 1520, in hasChanged
    return super(object, self).hasChanged(key)
  File "/usr/lib/python3/dist-packages/univention/admin/handlers/__init__.py", line 290, in hasChanged
    return not univention.admin.mapping.mapCmp(self.mapping, key, self.oldinfo.get(key, ''), self.info.get(key, ''))
  File "/usr/lib/python3/dist-packages/univention/admin/mapping.py", line 646, in mapCmp
    return f(old) == f(new)
  File "/usr/lib/python3/dist-packages/univention/admin/handlers/users/user.py", line 1230, in mapDateTimeTimezoneTupleToUTCDateTimeString
    dt = datetime_from_local_datetimetimezone_tuple(local_datetimetimezone_tuple)
  File "/usr/lib/python3/dist-packages/univention/admin/handlers/users/user.py", line 1224, in datetime_from_local_datetimetimezone_tuple
    naive_dt = datetime.strptime("%s %s" % (d, t), "%Y-%m-%d %H:%M")
  File "/usr/lib/python3.7/_strptime.py", line 577, in _strptime_datetime
    tt, fraction, gmtoff_fraction = _strptime(data_string, format)
  File "/usr/lib/python3.7/_strptime.py", line 362, in _strptime
    data_string[found.end():])
ValueError: unconverted data remains: :23

+++ This bug was initially created as a clone of Bug #53631 +++

Comment 1 Florian Best

2021-09-23 18:21:31 CEST

The syntax class "TimeString" which is used by ActivationDateTimeTimezone allows '00:00' and '23:59:59'.