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

PartialTrustDomain - Handle SecurityException to allow startup #1873

Merged
merged 1 commit into from
Dec 22, 2016

Conversation

snakefoot
Copy link
Contributor

@snakefoot snakefoot commented Dec 21, 2016

Might resolve #1870


This change is Reviewable

@304NotModified
Copy link
Member

Cool!

@304NotModified 304NotModified added the enhancement Improvement on existing feature label Dec 21, 2016
@304NotModified 304NotModified added this to the 4.4.1 milestone Dec 21, 2016
@304NotModified
Copy link
Member

Yeah missing file header warning :p

@codecov-io
Copy link

codecov-io commented Dec 22, 2016

Current coverage is 81% (diff: 72%)

Merging #1873 into master will increase coverage by <1%

@@             master      #1873   diff @@
==========================================
  Files           276        276          
  Lines         18601      18628    +27   
  Methods        2860       2861     +1   
  Messages          0          0          
  Branches       2138       2139     +1   
==========================================
+ Hits          15108      15147    +39   
+ Misses         2991       2978    -13   
- Partials        502        503     +1   

Sunburst

Powered by Codecov. Last update 5be4aa9...cd91764

@304NotModified 304NotModified merged commit 3cf67a8 into NLog:master Dec 22, 2016
@304NotModified
Copy link
Member

👍 👍

@snakefoot snakefoot deleted the AppDomainPartialTrustTest branch October 10, 2017 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvement on existing feature platform support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants