RE: [jira] [Commented] (LOG4NET-524) OnlyOnceErrorHandler might cause StackOverflowError

2016-08-27 Thread Arun Kumar 
Unsubscribe

Sent from my Windows Phone

From: Dominik Psenner (JIRA)<mailto:j...@apache.org>
Sent: ‎24-‎08-‎2016 22:39
To: log4net-dev@logging.apache.org<mailto:log4net-dev@logging.apache.org>
Subject: [jira] [Commented] (LOG4NET-524) OnlyOnceErrorHandler might cause 
StackOverflowError


[ 
https://issues.apache.org/jira/browse/LOG4NET-524?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15435303#comment-15435303
 ]

Dominik Psenner commented on LOG4NET-524:
-

Please subscribe to the log4j mailing list to track notifications and get 
involved. I do not actively hack on log4j and will most likely loose track of 
this.

> OnlyOnceErrorHandler might cause StackOverflowError
> ---
>
> Key: LOG4NET-524
> URL: https://issues.apache.org/jira/browse/LOG4NET-524
> Project: Log4net
>  Issue Type: Bug
>  Components: Core
>Affects Versions: 1.2.15
>Reporter: Marc Theisen
>Assignee: Dominik Psenner
>
> I am referring to the solution in 
> https://bz.apache.org/bugzilla/show_bug.cgi?id=42151.
> I am suggesting another approach instead of placing a bad hack into 
> com.{{csfb.fao.carat.util.file.LoggingOutputStream}}.
> Why not change the {{error}} method of class {{OnlyOnceErrorHandler}} from
> {code}
> public void error(String message, Exception e, int errorCode, LoggingEvent 
> event)
> {
>   if (e instanceof InterruptedIOException || e instanceof 
> InterruptedException)
>   {
> Thread.currentThread().interrupt();
>   }
>   if(firstTime)
>   {
> LogLog.error(message, e);
> firstTime = false;   // yikes, too late if stdout/stderr has been
>   }  // redirected
> }
> {code}
> to
> {code}
> public void error(String message, Exception e, int errorCode, LoggingEvent 
> event)
> {
>   if (e instanceof InterruptedIOException || e instanceof 
> InterruptedException)
>   {
> Thread.currentThread().interrupt();
>   }
>   if(firstTime)
>   {
> firstTime = false;
> LogLog.error(message, e);
>   }
> }
> {code}



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

[jira] [Commented] (LOG4NET-524) OnlyOnceErrorHandler might cause StackOverflowError

2016-08-24 Thread Dominik Psenner (JIRA) 

[ 
https://issues.apache.org/jira/browse/LOG4NET-524?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15435303#comment-15435303
 ] 

Dominik Psenner commented on LOG4NET-524:
-

Please subscribe to the log4j mailing list to track notifications and get 
involved. I do not actively hack on log4j and will most likely loose track of 
this.

> OnlyOnceErrorHandler might cause StackOverflowError
> ---
>
> Key: LOG4NET-524
> URL: https://issues.apache.org/jira/browse/LOG4NET-524
> Project: Log4net
>  Issue Type: Bug
>  Components: Core
>Affects Versions: 1.2.15
>Reporter: Marc Theisen
>Assignee: Dominik Psenner
>
> I am referring to the solution in 
> https://bz.apache.org/bugzilla/show_bug.cgi?id=42151.
> I am suggesting another approach instead of placing a bad hack into 
> com.{{csfb.fao.carat.util.file.LoggingOutputStream}}.
> Why not change the {{error}} method of class {{OnlyOnceErrorHandler}} from
> {code}
> public void error(String message, Exception e, int errorCode, LoggingEvent 
> event)
> {
>   if (e instanceof InterruptedIOException || e instanceof 
> InterruptedException)
>   {
> Thread.currentThread().interrupt();
>   }
>   if(firstTime)
>   {
> LogLog.error(message, e);
> firstTime = false;   // yikes, too late if stdout/stderr has been
>   }  // redirected
> }
> {code}
> to
> {code}
> public void error(String message, Exception e, int errorCode, LoggingEvent 
> event)
> {
>   if (e instanceof InterruptedIOException || e instanceof 
> InterruptedException)
>   {
> Thread.currentThread().interrupt();
>   }
>   if(firstTime)
>   {
> firstTime = false;
> LogLog.error(message, e);
>   }
> }
> {code}



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

[jira] [Commented] (LOG4NET-524) OnlyOnceErrorHandler might cause StackOverflowError

2016-08-24 Thread Marc Theisen (JIRA) 

[ 
https://issues.apache.org/jira/browse/LOG4NET-524?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15435282#comment-15435282
 ] 

Marc Theisen commented on LOG4NET-524:
--

Apologies for the wrong report, [~dpsen...@gmail.com]. Could you please post 
the link to the log4j bug once it has been assigned? Thanks!

> OnlyOnceErrorHandler might cause StackOverflowError
> ---
>
> Key: LOG4NET-524
> URL: https://issues.apache.org/jira/browse/LOG4NET-524
> Project: Log4net
>  Issue Type: Bug
>  Components: Core
>Affects Versions: 1.2.15
>Reporter: Marc Theisen
>Assignee: Dominik Psenner
>
> I am referring to the solution in 
> https://bz.apache.org/bugzilla/show_bug.cgi?id=42151.
> I am suggesting another approach instead of placing a bad hack into 
> com.{{csfb.fao.carat.util.file.LoggingOutputStream}}.
> Why not change the {{error}} method of class {{OnlyOnceErrorHandler}} from
> {code}
> public void error(String message, Exception e, int errorCode, LoggingEvent 
> event)
> {
>   if (e instanceof InterruptedIOException || e instanceof 
> InterruptedException)
>   {
> Thread.currentThread().interrupt();
>   }
>   if(firstTime)
>   {
> LogLog.error(message, e);
> firstTime = false;   // yikes, too late if stdout/stderr has been
>   }  // redirected
> }
> {code}
> to
> {code}
> public void error(String message, Exception e, int errorCode, LoggingEvent 
> event)
> {
>   if (e instanceof InterruptedIOException || e instanceof 
> InterruptedException)
>   {
> Thread.currentThread().interrupt();
>   }
>   if(firstTime)
>   {
> firstTime = false;
> LogLog.error(message, e);
>   }
> }
> {code}



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