----- Original Message ----- > From: "Einav Cohen" <[email protected]> > To: "engine-devel" <[email protected]>, "users" <[email protected]>, > "Martin Perina" <[email protected]> > Sent: Tuesday, January 14, 2014 11:23:34 AM > Subject: [Engine-devel] Showing stacktrace info in GUI error dialog > > Hi, this is about patch [1] - showing server-side-exception > stacktrace info in GUI error dialog. > > a couple of notes about this patch: > > (1) I find it a little strange that server-side exceptions > are not logged in the server side and are / will be displayed > only on the client-side - this needs to be fixed first. > +1.
> (2) once everything is logged on the server side, we can > debate whether it makes sense to put the full exception > stack-trace *also* in the GUI error pop-up. > I think that it is not completely necessary (since if there > is a server side exception, the user is likely to go to the > server side log anyway to get the full picture / context), but > if it is (and I'd like to hear your opinion about that) - we > shouldn't show the entire exception in the dialog once the dialog > is displayed - that's too much information to absorb. > We should show whatever we are showing today, and also something > like a "More Details" collapsible section (collapsed by default) > that, when expanded, displays the full exception details. > > comments are welcome. I personally appreciate web apps that allow me to see an error (or even expose an entire log file), but do so gracefully -- hide it under a option, use a "More Details" option like you mention, etc. I think saving a person the step of ssh'ing to a server and digging for a log file is a nice-to-have. Greg > > ---- > Thanks, > Einav > > [1] http://gerrit.ovirt.org/#/c/23096/ > _______________________________________________ > Engine-devel mailing list > [email protected] > http://lists.ovirt.org/mailman/listinfo/engine-devel > _______________________________________________ Users mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/users
