Univention Bugzilla – Bug 28627
Traceback bei UCS-ungültigen Benutzern im LDAP
Last modified: 2018-04-13 13:28:21 CEST
Traceback (most recent call last): File "/usr/share/univention-directory-manager-tools/univention-cli-server", line 233, in doit output = univention.admincli.admin.doit(arglist) File "/usr/lib/pymodules/python2.6/univention/admincli/admin.py", line 387, in doit out=_doit(arglist) File "/usr/lib/pymodules/python2.6/univention/admincli/admin.py", line 1107, in _doit out.append( 'DN: %s' % _2utf8( univention.admin.objects.dn (object ) ) ) File "/usr/lib/pymodules/python2.6/univention/admincli/admin.py", line 243, in _2utf8 return text.decode( 'iso-8859-1' ) AttributeError: 'NoneType' object has no attribute 'decode' def _2utf8( text ): try: return text.encode( 'utf-8' ) except: return text.decode( 'iso-8859-1' )
/usr/lib/pymodules/python2.6/univention/admin/objects.py:139 def dn(object): if hasattr(object, 'dn'): return object.dn else: return None
FYI: Das ganze ist im Rahmen von Ticket #2012081021002877 aufgetreten: Dort werden massenhaft Benutzer (inetOrgPerson) direkt im LDAP angelegt, deren UID mehr als einen Großbuchstaben am Anfang enthielten. Der LDAP-Suchfilter von users/user scheint die auch noch zu finden, allerdings geht wohl dann was beim Öffnen schief, so daß die Objekte alle mit DN=None angezeigt werden (bzw. eben besagter Traceback)
UMC verschluckt sich auch irgendwo und steigt mit einem Traceback aus: File '/usr/lib/pymodules/python2.6/notifier/threads.py', line 82, in _run tmp = self._function() File '/usr/lib/pymodules/python2.6/notifier/__init__.py', line 104, in __call__ return self._function( *tmp, **self._kwargs ) File '/usr/lib/pymodules/python2.6/univention/management/console/modules/udm/__init__.py', line 426, in _thread if module.module is None: AttributeError: 'NoneType' object has no attribute 'module' /usr/lib/pymodules/python2.6/univention/management/console/modules/udm/__init__.py > module = get_module( object_type, obj.dn ) > if module.module is None: Sollte das nicht besser nur "module" heißen? > MODULE.warn( 'Could not identify LDAP object %s (flavor: %s). The object is ignored.' % ( obj.dn, request.flavor ) ) > continue
(In reply to comment #3) > ... > Sollte das nicht besser nur "module" heißen? Nein, das ist wahrscheinlich schon richtig. Es gibt noch eine Klasse, die die UDM-Module wrapt. Insofern greift module.module auf das eigentliche UDM-Modul zu, welches als Eigenschaft "module" an dem Wrapper-Objekt "module" gespeichert ist :) .
Siehe Ticket 2012081021002877 für ein (vermutlich) weiteres Problem, das ich auch lokal selber reproduzieren kann: ---------- File '/usr/lib/pymodules/python2.6/notifier/threads.py', line 82, in _run tmp = self._function() File '/usr/lib/pymodules/python2.6/notifier/__init__.py', line 104, in __call__ return self._function( *tmp, **self._kwargs ) File '/usr/lib/pymodules/python2.6/univention/management/console/modules/udm/__init__.py', line 856, in _thread 'path' : ldap_dn2path( obj.dn, include_rdn = False ) } ) File '/usr/lib/pymodules/python2.6/univention/management/console/modules/udm/udm_ldap.py', line 827, in ldap_dn2path if ldap_base is None or not ldap_dn.endswith( ldap_base ): ^^^^ ^^ ^^^^ AttributeError: 'NoneType' object has no attribute 'endswith' ---------- univention-management-console-frontend 1.0.396-1.366.201207161556 (24595) Der Code wurde durch Bug #22439 in svn 25026 hinzugefügt. Auch sehr unglücklich ist folgender Code in admin/handlers/users/user.py#__init__(): > if uid: > try: ... > # FIXME: we should NEVER catch all exceptions > except Exception, e: > # at least write some debuging output.. > univention.debug.debug(univention.debug.ADMIN, univention.debug.INFO, 'Caught exception: %s' % e ) > univention.debug.debug(univention.debug.ADMIN, univention.debug.INFO, 'Continuing without dn..') > self.dn=None > return Das Problem ist hier, daß die Exception lediglich ab level=INFO geloggt wird. Da standardmäßig das Level niedrigr eingestellt ist, sieht man von dem Problem nichts direkt, sondern danach lediglich die Fehlermeldung, daß dn=None nicht das Attribut endwidth hat. IMHO sollte das als Error geloggt werden. (In reply to comment #4) > (In reply to comment #3) > > ... > > Sollte das nicht besser nur "module" heißen? > > Nein, das ist wahrscheinlich schon richtig. Das sollte es wohl besser if module is None or module.module is None: heißen.
(In reply to comment #5) > Auch sehr unglücklich ist folgender Code in ... > Das Problem ist hier, daß die Exception lediglich ab level=INFO geloggt wird. > Da standardmäßig das Level niedrigr eingestellt ist, sieht man von dem Problem > nichts direkt, sondern danach lediglich die Fehlermeldung, daß dn=None nicht > das Attribut endwidth hat. > IMHO sollte das als Error geloggt werden. Auslöser ist ein vergessenes Aufruf von set_uid_umlaut(), der hier einen AttributeError auslöst; siehe dazu Bug #28671
This issue has been filed against UCS 3. UCS 3 is out of the normal maintenance and many UCS components have vastly changed in UCS 4. If this issue is still valid, please change the version to a newer UCS version otherwise this issue will be automatically closed in the next weeks.
There is a Customer ID set so I set the flag "Enterprise Customer affected".
This issue has been filed against UCS 3.0. UCS 3.0 is out of maintenance and many UCS components have vastly changed in later releases. Thus, this issue is now being closed. If this issue still occurs in newer UCS versions, please use "Clone this bug" or reopen this issue. In this case please provide detailed information on how this issue is affecting you.