Red Hat Bugzilla – Bug 824898
Message center details view 'hides' real cause
Last modified: 2013-09-01 05:57:45 EDT
I tried to add an as7 manually in the release branch and it failed with the red
bar. So I wen to the message center to see what is going on.
Remember that I have a 1920*x screen. So pretty wide. And still I had to scroll
8k pixels to the right in order to see the
real message that the plugin wanted to tell me.
For the user this is a usability issue.
We should in the first "Details" line only show the message from the inner most
exception (what is on the right side of the screen)
And then lower down (after some newlines) show what we have today.
Bonus points for replacing each '->' in the original details line with a line
break (or adding the line break before '->'
Created attachment 586649 [details]
Leftmost part of the output
Created attachment 586650 [details]
Rightmost part of the message showing the real info
I will have a "Root Cause" show in the details window that shows you the inner-most exception message (minus the throwable class name). You will also have the option to see the Root Cause from the right-mouse menu that shows on the message bar.
If there is no root cause (meaning, if there is no exceptions - its just a "normal" message), then there will be no root cause displayed and the menu item to show the root cause will be disabled.
git commit to master: 325170b9e32037b748bf2ef12c0954f9d6f35ad2
(another git commit that tweeked the test page is b8ababacc8dbd7ee3d6ebbf3681f63da3334d44c )
I made some commits to master that aren't directly addressing this issue, but came out of researching this stuff:
802ff8d372ca0431c96a7c735021790ec0c0d1c0 - fixes the ThrowableUtil utility so it will as least record the exception class name if the message is null (better to show the exception class name than just the string "null")
b8f67855445081683f56bd1dd27e0239f7114a9e - fix the logging done on the GWT service impl so it logs the full server-side exception before getting munged and sent over to the gwt client
Setting Target Release to be RHQ4.5, I'll clone this for something to get into JON3.1.1
git commit to master: 536e094
this code has the gwt client look for the ThrowableUtil marker (which is " -> ") in the exception message - if it sees it, it will get the last message after the last -> marker and use that as the root cause string
all code is in master. these are the related commits to try to get the root cause to be displayed in the message center
there can be additional UI improvements (should we show a little icon that you click to popup the context menu in the message bar? should we show some root cause string in the bar itself? (be careful, the root cause can be a long string), etc.). We will write up additional BZs for additional UI improvements for the message center. This BZ is being moved to ON_QA since it addresses the question at hand - that being, show the root cause string for the inner most exception that caused the message to be displayed.
Bulk closing of items that are on_qa and in old RHQ releases, which are out for a long time and where the issue has not been re-opened since.