Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ConsoleTarget - DetectConsoleAvailable - Disabled by default #1778

Merged
merged 1 commit into from
Nov 22, 2016

Conversation

snakefoot
Copy link
Contributor

@snakefoot snakefoot commented Nov 21, 2016

Resolves #1773

DetectConsoleAvailable fails to handle the situation where the console application stdout is used for file logging, and the console application runs as a Windows Service or within the Task Scheduler.

DetectConsoleAvailable has been designed to disable the console, when the console application is running as windows service. And not for doing the funky chicken with stdout support.


This change is Reviewable

@304NotModified
Copy link
Member

Thanks again @snakefoot !

@304NotModified 304NotModified added enhancement Improvement on existing feature console-target labels Nov 21, 2016
@304NotModified 304NotModified added this to the 4.4 milestone Nov 21, 2016
@codecov-io
Copy link

Current coverage is 80% (diff: 100%)

Merging #1778 into master will decrease coverage by <1%

@@             master      #1778   diff @@
==========================================
  Files           276        277     +1   
  Lines         17711      17780    +69   
  Methods        2768       2785    +17   
  Messages          0          0          
  Branches       2020       2024     +4   
==========================================
+ Hits          14271      14298    +27   
- Misses         2987       3031    +44   
+ Partials        453        451     -2   

Sunburst

Powered by Codecov. Last update 00e58af...f23d8b0

@304NotModified 304NotModified merged commit 0fba1d9 into NLog:master Nov 22, 2016
@304NotModified 304NotModified added the documentation done all docs done (wiki, api docs, lists on nlog-project.org, xmldocs) label Nov 24, 2016
@304NotModified
Copy link
Member

@snakefoot snakefoot deleted the ConsoleTargetDetection branch October 10, 2017 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
console-target documentation done all docs done (wiki, api docs, lists on nlog-project.org, xmldocs) enhancement Improvement on existing feature needs documentation on wiki
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants