Univention Bugzilla – Bug 27940
Fehlerausgabe von UDM-Exceptions
Last modified: 2013-02-15 17:51:02 CET
Derzeit werden UDM-Syntax-Fehlermeldungen von UCS@schoo-UMC-Modulen doppelt ausgegen: ==================== Das Zurücksetzen des Passworts ist für die folgenden Benutzer fehlgeschlagen: * Krause, Daniel (d.krause1) Der Domänenteil der primären Mailadresse ist nicht in der Liste der eingerichteten Maildomänen enthalten: d.krause1@school.qa Der Domänenteil der primären Mailadresse ist nicht in der Liste der eingerichteten Maildomänen enthalten: d.krause1@school.qa ==================== Die Korrekturen wurden zu 3.0-2 durchgeführt, sie dazu auch Bug 7389, Comment 25 sowie Bug 24841.
Hier bitte mal schauen, ob ein Errata-Update dafür notwendig ist, oder ob es sich dabei um einen Sonderfall handelt.
Der hier beschriebene Bug tritt nicht bei allen Fehlermeldungen auf, sondern nur bei ganz bestimmten. Da der Bug weder das System gefährdet noch den Benutzer in der Bedienbarkeit einschränkt, würde ich sagen, dass es an dieser Stelle reicht den auf UCS 3.1 zu setzen.
(In reply to comment #2) > ... > Da der Bug weder das System gefährdet noch den Benutzer in der Bedienbarkeit > einschränkt, würde ich sagen, dass es an dieser Stelle reicht den auf UCS 3.1 > zu setzen. OK
Die doppelte Fehlermeldungsausgabe in ucs-school-umc-users wurde gefixt. Weiter konnte ich nach mehreren Versuchen mit grep nicht finden. ucs-school-umc-users (9.0.1-1)
Der Fix ist vielleicht zu vereinfachend. Jetzt wird nämlich (e.message or '\n'.join(e)) zurückgegeben, also nur eine von beiden. Es gibt aber Fehler, die brauchen beides. univention.admin.uexceptions.prohibitedUsername etwa - zugegebenermaßen das schlechtestmögliche Beispiel: "* <username>: Prohibited username. : <username>" wird nun zu "* <username>: Prohibited username.". Ich würde sogar behaupten, so ist es besser. Aber es sind eben nicht alle Informationen dabei, die mitgeliefert wurden. Bei anderen Exceptions kann das kritischer sein. Vielleicht nicht die paar, die realistischerweise beim Zurücksetzen des Passwortes auftreten, aber vielleicht ändert sich das mal. Sehr viel besser wäre es, folgende Funktion zu benutzen: udm.udm_ldap.get_exception_msg(e) Existiert die Abhängigkeit von UDM irgendwo in UCS@school? Wenn nicht, spricht etwas dagegen?
(In reply to comment #5) > Existiert die Abhängigkeit von UDM irgendwo in UCS@school? Wenn nicht, spricht > etwas dagegen? Ich glaube, UDM ist nicht zwingend notwendig. In diesem Fall einfach folgendes machen: 1. Funktion von udm_ldap.py kopieren und nutzen. FIXME: dranschreiben 2. Bug gegen UCS 3.2 eröffnen: udm_ldap.get_exception_msg(e) soll in admin.uexceptions wandern. Außerdem sollte das die Implementierung von __str__ für die alle Exceptions werden 3. Bug gegen UCS@school 3.2 eröffnen: Die kopierte Funktion wieder entfernen und str(e) zurückgeben.
(In reply to comment #6) > (In reply to comment #5) > > Existiert die Abhängigkeit von UDM irgendwo in UCS@school? Wenn nicht, spricht > > etwas dagegen? > > Ich glaube, UDM ist nicht zwingend notwendig. In diesem Fall einfach folgendes > machen: > 1. Funktion von udm_ldap.py kopieren und nutzen. FIXME: dranschreiben > 2. Bug gegen UCS 3.2 eröffnen: udm_ldap.get_exception_msg(e) soll in > admin.uexceptions wandern. Außerdem sollte das die Implementierung von __str__ > für die alle Exceptions werden > 3. Bug gegen UCS@school 3.2 eröffnen: Die kopierte Funktion wieder entfernen > und str(e) zurückgeben. Ok, ucs-school-umc-users (9.0.3-1), Bug #30089, Bug #30088
Funktioniert. Bugs wurden eröffnet. Changelog Ok
UCS@school 3.1 has been released: http://forum.univention.de/viewtopic.php?f=26&t=2364 If this error occurs again, please use "Clone This Bug".