Univention Bugzilla – Full Text Bug Listing |
Summary: | Fehlerausgabe von UDM-Exceptions | ||
---|---|---|---|
Product: | UCS@school | Reporter: | Alexander Kläser <klaeser> |
Component: | UMC | Assignee: | Florian Best <best> |
Status: | CLOSED FIXED | QA Contact: | Dirk Wiesenthal <wiesenthal> |
Severity: | minor | ||
Priority: | P3 | CC: | wiesenthal |
Version: | UCS@school 3.0 | ||
Target Milestone: | UCS@school 3.1 | ||
Hardware: | Other | ||
OS: | Linux | ||
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: | |||
Bug Depends on: | |||
Bug Blocks: | 30088, 30089 |
Description
Alexander Kläser
2012-07-16 17:20:57 CEST
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". |