[jira] [Commented] (LOG4NET-437) LogicalThreadContext data does not flow properly using Async and Await

2015-03-31 Thread Mike Barry (JIRA)

[ 
https://issues.apache.org/jira/browse/LOG4NET-437?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14388407#comment-14388407
 ] 

Mike Barry commented on LOG4NET-437:


Thanks. This solves the issue!

 LogicalThreadContext data does not flow properly using Async and Await
 --

 Key: LOG4NET-437
 URL: https://issues.apache.org/jira/browse/LOG4NET-437
 Project: Log4net
  Issue Type: Bug
  Components: Core
Affects Versions: 1.2.9, 1.2.10, 1.2.11, 1.2.12, 1.2.13
 Environment: Windows 7, .NET 4.5
Reporter: Mike Barry
 Attachments: LogicalThreadContext.diff, Program.cs, 
 ThreadContextStacks.diff


 LogicalThreadContext doesn't work properly when using Async and Await. The 
 current NDC call stack gets corrupted because the logical data context is not 
 immutable. (See: 
 http://blog.stephencleary.com/2013/04/implicit-async-context-asynclocal.html 
 for the gory details). I've attached a Program.cs that demonstrates the 
 issue. You will see with an unpatched log4net the thread context is corrupted 
 compared to the proper implementation in MyStack and the console logger. An 
 entry like:
 LOG4NET Main 1 A 2 A: /MoreWork
 LOG4NET Main 1 A 2 A: /MoreWork
 is invalid and should not be possible in a properly working thread context.
 I've attached a VERY naive patch that corrects the issue.



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


RE: [jira] [Commented] (LOG4NET-437) LogicalThreadContext data does not flow properly using Async and Await

2015-03-30 Thread Alexander Houben
You probably mean LOG4NET-455.

 -Original Message-
 From: Stefan Bodewig (JIRA) [mailto:j...@apache.org]
 Sent: Montag, 30. März 2015 08:03
 To: log4net-dev@logging.apache.org
 Subject: [jira] [Commented] (LOG4NET-437) LogicalThreadContext data
 does not flow properly using Async and Await
 
 
 [ https://issues.apache.org/jira/browse/LOG4NET-
 437?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-
 tabpanelfocusedCommentId=14386242#comment-14386242 ]
 
 Stefan Bodewig commented on LOG4NET-437:
 
 
 Sorry, when I opened LOG4NET-456 I completely forgot about this one.
 Could you please verify the fix applied for LOG4NET-456 works for your
 application as well?
 
  LogicalThreadContext data does not flow properly using Async and
 Await
  -
 -
 
  Key: LOG4NET-437
  URL: https://issues.apache.org/jira/browse/LOG4NET-
 437
  Project: Log4net
   Issue Type: Bug
   Components: Core
 Affects Versions: 1.2.9, 1.2.10, 1.2.11, 1.2.12, 1.2.13
  Environment: Windows 7, .NET 4.5
 Reporter: Mike Barry
  Attachments: LogicalThreadContext.diff, Program.cs,
 ThreadContextStacks.diff
 
 
  LogicalThreadContext doesn't work properly when using Async and
 Await. The current NDC call stack gets corrupted because the logical
 data context is not immutable. (See:
 http://blog.stephencleary.com/2013/04/implicit-async-context-
 asynclocal.html for the gory details). I've attached a Program.cs that
 demonstrates the issue. You will see with an unpatched log4net the
 thread context is corrupted compared to the proper implementation in
 MyStack and the console logger. An entry like:
  LOG4NET Main 1 A 2 A: /MoreWork
  LOG4NET Main 1 A 2 A: /MoreWork
  is invalid and should not be possible in a properly working thread
 context.
  I've attached a VERY naive patch that corrects the issue.
 
 
 
 --
 This message was sent by Atlassian JIRA
 (v6.3.4#6332)


smime.p7s
Description: S/MIME cryptographic signature


[jira] [Commented] (LOG4NET-437) LogicalThreadContext data does not flow properly using Async and Await

2015-03-30 Thread Stefan Bodewig (JIRA)

[ 
https://issues.apache.org/jira/browse/LOG4NET-437?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14386242#comment-14386242
 ] 

Stefan Bodewig commented on LOG4NET-437:


Sorry, when I opened LOG4NET-456 I completely forgot about this one.  Could you 
please verify the fix applied for LOG4NET-456 works for your application as 
well?

 LogicalThreadContext data does not flow properly using Async and Await
 --

 Key: LOG4NET-437
 URL: https://issues.apache.org/jira/browse/LOG4NET-437
 Project: Log4net
  Issue Type: Bug
  Components: Core
Affects Versions: 1.2.9, 1.2.10, 1.2.11, 1.2.12, 1.2.13
 Environment: Windows 7, .NET 4.5
Reporter: Mike Barry
 Attachments: LogicalThreadContext.diff, Program.cs, 
 ThreadContextStacks.diff


 LogicalThreadContext doesn't work properly when using Async and Await. The 
 current NDC call stack gets corrupted because the logical data context is not 
 immutable. (See: 
 http://blog.stephencleary.com/2013/04/implicit-async-context-asynclocal.html 
 for the gory details). I've attached a Program.cs that demonstrates the 
 issue. You will see with an unpatched log4net the thread context is corrupted 
 compared to the proper implementation in MyStack and the console logger. An 
 entry like:
 LOG4NET Main 1 A 2 A: /MoreWork
 LOG4NET Main 1 A 2 A: /MoreWork
 is invalid and should not be possible in a properly working thread context.
 I've attached a VERY naive patch that corrects the issue.



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


Re: [jira] [Commented] (LOG4NET-437) LogicalThreadContext data does not flow properly using Async and Await

2015-03-30 Thread Stefan Bodewig
On 2015-03-30, Alexander Houben wrote:

 You probably mean LOG4NET-455.

Yes, thanks!

Stefan


signature.asc
Description: PGP signature


[jira] [Commented] (LOG4NET-437) LogicalThreadContext data does not flow properly using Async and Await

2015-03-30 Thread Mike Barry (JIRA)

[ 
https://issues.apache.org/jira/browse/LOG4NET-437?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14387371#comment-14387371
 ] 

Mike Barry commented on LOG4NET-437:


Build fails in code from github. The files do not exist: 
https://github.com/jabbera/log4net/tree/trunk/src/log4net/Config

1CSC : error CS2001: Source file 'Config\AliasDomainAttribute.cs' could not be 
found
1CSC : error CS2001: Source file 'Config\DomainAttribute.cs' could not be found
1CSC : error CS2001: Source file 'Config\DOMConfigurator.cs' could not be found
1CSC : error CS2001: Source file 'Config\DOMConfiguratorAttribute.cs' could 
not be found

 LogicalThreadContext data does not flow properly using Async and Await
 --

 Key: LOG4NET-437
 URL: https://issues.apache.org/jira/browse/LOG4NET-437
 Project: Log4net
  Issue Type: Bug
  Components: Core
Affects Versions: 1.2.9, 1.2.10, 1.2.11, 1.2.12, 1.2.13
 Environment: Windows 7, .NET 4.5
Reporter: Mike Barry
 Attachments: LogicalThreadContext.diff, Program.cs, 
 ThreadContextStacks.diff


 LogicalThreadContext doesn't work properly when using Async and Await. The 
 current NDC call stack gets corrupted because the logical data context is not 
 immutable. (See: 
 http://blog.stephencleary.com/2013/04/implicit-async-context-asynclocal.html 
 for the gory details). I've attached a Program.cs that demonstrates the 
 issue. You will see with an unpatched log4net the thread context is corrupted 
 compared to the proper implementation in MyStack and the console logger. An 
 entry like:
 LOG4NET Main 1 A 2 A: /MoreWork
 LOG4NET Main 1 A 2 A: /MoreWork
 is invalid and should not be possible in a properly working thread context.
 I've attached a VERY naive patch that corrects the issue.



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


[jira] [Commented] (LOG4NET-437) LogicalThreadContext data does not flow properly using Async and Await

2015-03-30 Thread Stefan Bodewig (JIRA)

[ 
https://issues.apache.org/jira/browse/LOG4NET-437?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14387959#comment-14387959
 ] 

Stefan Bodewig commented on LOG4NET-437:


Oops, my fault.  I deleted the classes but didn't update the VS files (I'm 
using NAnt to build log4net).  Fixed now.

 LogicalThreadContext data does not flow properly using Async and Await
 --

 Key: LOG4NET-437
 URL: https://issues.apache.org/jira/browse/LOG4NET-437
 Project: Log4net
  Issue Type: Bug
  Components: Core
Affects Versions: 1.2.9, 1.2.10, 1.2.11, 1.2.12, 1.2.13
 Environment: Windows 7, .NET 4.5
Reporter: Mike Barry
 Attachments: LogicalThreadContext.diff, Program.cs, 
 ThreadContextStacks.diff


 LogicalThreadContext doesn't work properly when using Async and Await. The 
 current NDC call stack gets corrupted because the logical data context is not 
 immutable. (See: 
 http://blog.stephencleary.com/2013/04/implicit-async-context-asynclocal.html 
 for the gory details). I've attached a Program.cs that demonstrates the 
 issue. You will see with an unpatched log4net the thread context is corrupted 
 compared to the proper implementation in MyStack and the console logger. An 
 entry like:
 LOG4NET Main 1 A 2 A: /MoreWork
 LOG4NET Main 1 A 2 A: /MoreWork
 is invalid and should not be possible in a properly working thread context.
 I've attached a VERY naive patch that corrects the issue.



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