Bug 27940 - Fehlerausgabe von UDM-Exceptions
Fehlerausgabe von UDM-Exceptions
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: UMC
UCS@school 3.0
Other Linux
: P3 minor (vote)
: UCS@school 3.1
Assigned To: Florian Best
Dirk Wiesenthal
:
Depends on:
Blocks: 30088 30089
  Show dependency treegraph
 
Reported: 2012-07-16 17:20 CEST by Alexander Kläser
Modified: 2013-02-15 17:51 CET (History)
1 user (show)

See Also:
What kind of report is it?: ---
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): Usability
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Kläser univentionstaff 2012-07-16 17:20:57 CEST
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.
Comment 1 Alexander Kläser univentionstaff 2012-08-01 10:04:44 CEST
Hier bitte mal schauen, ob ein Errata-Update dafür notwendig ist, oder ob es sich dabei um einen Sonderfall handelt.
Comment 2 Jascha Geerds univentionstaff 2012-08-02 14:34:58 CEST
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.
Comment 3 Alexander Kläser univentionstaff 2012-08-03 11:04:31 CEST
(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
Comment 4 Florian Best univentionstaff 2013-01-09 12:45:29 CET
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)
Comment 5 Dirk Wiesenthal univentionstaff 2013-01-17 20:06:14 CET
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?
Comment 6 Dirk Wiesenthal univentionstaff 2013-01-18 11:01:56 CET
(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.
Comment 7 Florian Best univentionstaff 2013-01-21 12:22:17 CET
(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
Comment 8 Dirk Wiesenthal univentionstaff 2013-01-21 14:21:59 CET
Funktioniert. Bugs wurden eröffnet. Changelog Ok
Comment 9 Sönke Schwardt-Krummrich univentionstaff 2013-02-15 17:51:02 CET
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".