Bug 25675 - Tracebacks in UMC
Tracebacks in UMC
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UNSTABLE
Other Linux
: P3 normal (vote)
: UCS 3.0-1
Assigned To: Florian Best
Jascha Geerds
:
: 25480 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-01-03 09:58 CET by Andreas Büsching
Modified: 2012-03-04 14:34 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 Andreas Büsching univentionstaff 2012-01-03 09:58:56 CET
Tracebacks sind für den Benutzer grundsätzlich nicht hilfreich und sollten in der Standardansicht auf die "wesentlichen" Informationen beschränkt werden. Es sollte weitehrin die Möglichkeit geben and die Erweiterten Informationen heranzukommen.

Beispiel:

Traceback (most recent call last):
File '/usr/lib/pymodules/python2.6/univention/management/console/modules/__init__.py', line 97, in execute
func( request )
File '/usr/lib/pymodules/python2.6/univention/management/console/modules/udm/udm_ldap.py', line 111, in wrapper_func
lo = udm_uldap.access( host = ucr.get( 'ldap/master' ), base = ucr.get( 'ldap/base' ), binddn = _user_dn, bindpw = _password )
File '/usr/lib/pymodules/python2.6/univention/admin/uldap.py', line 256, in __init__
raise univention.admin.uexceptions.authFail, _( 'Authentication failed' )
authFail: Authentisierung fehlgeschlagen


Für den Benutzer wäre hier die letzte Zeile ausreichend.

Ausserdem sollten die Tracebacks wieder an feedback@ geschickt werden können.
Comment 1 Alexander Kläser univentionstaff 2012-01-24 13:20:08 CET
In einem ersten Schritt sollte dies zunächst nur auf der Client-Seite umgesetzt werden (d.h. via Parsen der Fehlermeldung). Bitte dann einen neuen Bug anlegen, damit das UMC-Protokoll für Tracebacks zu einem späteren Zeitpunkt erweitert wird (dies würde zunächst zu viel Aufwand nach sich ziehen).
Comment 2 Florian Best univentionstaff 2012-01-30 11:42:25 CET
Tracebacks werden nun ausklappbar und werden standardmäßig nicht angezeigt.
Außerdem ist ein mailto-Link hinzugefügt worden mit einem kleinem Formular, in dem der Traceback, kurze Fragen zum Fehler sowie die umc-Version steht.

Das Paket ist gebaut, Changelog gemacht.
Comment 3 Florian Best univentionstaff 2012-01-31 15:34:05 CET
(In reply to comment #1)
> In einem ersten Schritt sollte dies zunächst nur auf der Client-Seite umgesetzt
> werden (d.h. via Parsen der Fehlermeldung). Bitte dann einen neuen Bug anlegen,
> damit das UMC-Protokoll für Tracebacks zu einem späteren Zeitpunkt erweitert
> wird (dies würde zunächst zu viel Aufwand nach sich ziehen).
Dafür habe ich Bug #25987 angelegt
Comment 4 Alexander Kläser univentionstaff 2012-02-03 12:57:57 CET
*** Bug 25480 has been marked as a duplicate of this bug. ***
Comment 5 Alexander Kläser univentionstaff 2012-02-09 18:59:14 CET
Mir ist gerade aufgefallen, dass in Chromium ein Traceback nicht richtig geparset wurde, da ein JSON-Objekt mit Zeilenumbrüchen in einem String nicht gültig sind, also bspw.:

{ "status": "591 ", "message": "Execution of command 'schoolrooms/get' has failed:x
Traceback (most recent call last):
  File '/usr/lib/pymodules/python2.6/univention/management/console/modules/__init__.py', line 97, in execute
    func( request )
  File '/usr/lib/pymodules/python2.6/univention/management/console/modules/schoolrooms/__init__.py', line 109, in get
    MODULE.info( 'schoolrooms.get: results: %s' % str( result ) )
NameError: global name 'result' is not defined
" }

Die Funktion umc.tools.parseError() soltle dies berücksichtigen, d.h. Zeilenumbrüche sollten durch ein anderes Zeichen ersetzt werden.
Comment 6 Florian Best univentionstaff 2012-02-10 14:53:56 CET
Änderungen gemacht

univention-management-console-frontend (1.0.260-1) unstable; urgency=low
  * added isEqual function to umc.tools (Bug #25631)
  * fixed error in json parsing caused by new lines in strings (Bug #25675)
Comment 7 Florian Best univentionstaff 2012-02-15 17:24:56 CET
Die Tracebacks werden jetzt zusätzlich beim ein und ausklappen zentriert.
Comment 8 Jascha Geerds univentionstaff 2012-02-21 14:17:19 CET
Bug wurde gefixed; Changelog vorhanden.
Comment 9 Sönke Schwardt-Krummrich univentionstaff 2012-03-04 14:34:15 CET
UCS 3.0-1 wurde veröffentlicht. Sollte der hier beschriebene Bug mit einer
neueren Version von UCS erneut auftreten, so sollte dieser Bug dupliziert
werden: "Clone This Bug"