Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Failing to lookup ProcessName because of Access Denied should fallback to Win32-API - #4182

#4182
Merged
304NotModified merged 1 commit into
NLog:masterNLog/NLog:masterfrom
snakefoot:ProcessNameNativeFallbacksnakefoot/NLog:ProcessNameNativeFallbackCopy head branch name to clipboard
Dec 6, 2020
Merged

Failing to lookup ProcessName because of Access Denied should fallback to Win32-API#4182
304NotModified merged 1 commit into
NLog:masterNLog/NLog:masterfrom
snakefoot:ProcessNameNativeFallbacksnakefoot/NLog:ProcessNameNativeFallbackCopy head branch name to clipboard

Conversation

@snakefoot

@snakefoot snakefoot commented Nov 25, 2020

Copy link
Copy Markdown
Contributor

Trying to resolve #4181 where IIS-applications with medium-trust no longer can use ${processname} and ${processid}

Bug introduced with #3941 in NLog 4.7.1. Guess there was a reason for using Win32 API besides speed optimization :)

@snakefoot snakefoot added the bug Bug report / Bug fix label Nov 25, 2020
@snakefoot snakefoot added this to the 4.7.6 milestone Nov 25, 2020
@snakefoot
snakefoot changed the base branch from dev to master November 25, 2020 21:32
@snakefoot snakefoot closed this Nov 25, 2020
@snakefoot snakefoot reopened this Nov 25, 2020
@snakefoot
snakefoot force-pushed the ProcessNameNativeFallback branch 2 times, most recently from f24dc4a to 5931a51 Compare November 25, 2020 21:48
@snakefoot
snakefoot force-pushed the ProcessNameNativeFallback branch from 5931a51 to 7e1a5f9 Compare November 26, 2020 08:02
@snakefoot snakefoot changed the title Failing to lookup ProcessName because of SecurityException should fallback to Win32-API Failing to lookup ProcessName because of Access Denied should fallback to Win32-API Nov 26, 2020
@sonarqubecloud

Copy link
Copy Markdown

@snakefoot

Copy link
Copy Markdown
Contributor Author

@304NotModified Ready for review. Gonna create some conflicts when merging to dev-branch, but that is another story.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bug report / Bug fix size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

IIS appliction running medium trust outputs ProcessName as Unknown

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.