tomee-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Romain Manni-Bucau (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (TOMEE-1619) Add exception message to Alerts
Date Sat, 01 Aug 2015 07:27:04 GMT

    [ https://issues.apache.org/jira/browse/TOMEE-1619?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14650197#comment-14650197
] 

Romain Manni-Bucau commented on TOMEE-1619:
-------------------------------------------

There are several cases of alerts but the most common is the webapp one so if it is not shown
in the gui then the info is almost useless.

Side note: there is a thread speaking of removing drop in war so this would almost be useless
to keep this class. Note sure the status anymore but surely something to check before working
on it.

About container log: when running it no guarantee you have it - even a container - so stdout
is an acceptable default.

> Add exception message to Alerts
> -------------------------------
>
>                 Key: TOMEE-1619
>                 URL: https://issues.apache.org/jira/browse/TOMEE-1619
>             Project: TomEE
>          Issue Type: Improvement
>          Components: TomEE Core Server
>    Affects Versions: 7.0.0
>            Reporter: Alex
>            Priority: Trivial
>              Labels: newbie
>             Fix For: 7.0.0
>
>
> The method org.apache.tomee.installer.Alerts.addError(String, Exception) only logs the
passed message, but not the exception message.
> This is tricky, because you don't know which exception in the chain actually carries
a useful message. I think just Exception.getMessage() is the best we can do?
> Also I noticed this is only performing an stdout. Should we also write to the container
log?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message