[ 
https://issues.apache.org/jira/browse/PDFBOX-1503?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Juraj Lonc updated PDFBOX-1503:
-------------------------------

    Description: 
I made web application which uses pdfbox library and its funcionality.
This web application is deployed on jboss. (log4j is used for logging)

If some exception occurs in pdfbox then exception is printed twice. That is not 
good. It makes mess in log and it is hard to use SMTPAppender or other appender 
that processes log events.

Problem is that you are using everywhere construction:

try {
...
} catch( Exception e ) {
 e.printStackTrace();
 LOG.error(e, e);
}

I think it would be nice to have it like this:

try {
...
} catch( Exception e ) {
 LOG.error(e, e);
}


  was:
I made web application which uses pdfbox library and its funcionality.
This web application is deployed on jboss. (log4j is used for logging)

If some exception occurs in pdfbox then exception is printed twice that is not 
good. It makes mess and it is hard to use SMTPAppender or other appender that 
processes log events.

Problem is that you are using everywhere construction:

try {
...
} catch( Exception e ) {
 e.printStackTrace();
 LOG.error(e, e);
}

I think it would be nice to have it like this:

try {
...
} catch( Exception e ) {
 LOG.error(e, e);
}


    
> double logging of exceptions
> ----------------------------
>
>                 Key: PDFBOX-1503
>                 URL: https://issues.apache.org/jira/browse/PDFBOX-1503
>             Project: PDFBox
>          Issue Type: Improvement
>            Reporter: Juraj Lonc
>
> I made web application which uses pdfbox library and its funcionality.
> This web application is deployed on jboss. (log4j is used for logging)
> If some exception occurs in pdfbox then exception is printed twice. That is 
> not good. It makes mess in log and it is hard to use SMTPAppender or other 
> appender that processes log events.
> Problem is that you are using everywhere construction:
> try {
> ...
> } catch( Exception e ) {
>  e.printStackTrace();
>  LOG.error(e, e);
> }
> I think it would be nice to have it like this:
> try {
> ...
> } catch( Exception e ) {
>  LOG.error(e, e);
> }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to