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

ConfigurationItemFactory - Skip assembly-loading should also check prefix-option - #5257

#5257
Merged
snakefoot merged 1 commit into
NLog:devNLog/NLog:devfrom
snakefoot:assembly-prefix-not-skipsnakefoot/NLog:assembly-prefix-not-skipCopy head branch name to clipboard
Jun 14, 2023
Merged

ConfigurationItemFactory - Skip assembly-loading should also check prefix-option#5257
snakefoot merged 1 commit into
NLog:devNLog/NLog:devfrom
snakefoot:assembly-prefix-not-skipsnakefoot/NLog:assembly-prefix-not-skipCopy head branch name to clipboard

Conversation

@snakefoot

@snakefoot snakefoot commented Jun 14, 2023

Copy link
Copy Markdown
Contributor

Resolves #5256 - Bug introduced with NLog v5.2 by #5191

NLog v5.2 introduced an optimization to skip assembly-reflection, when already loaded the recommended way, but it failed to recognize the use of custom assembly-prefix

@snakefoot snakefoot added the bug Bug report / Bug fix label Jun 14, 2023
@snakefoot
snakefoot force-pushed the assembly-prefix-not-skip branch 2 times, most recently from d7cee47 to da5fa2a Compare June 14, 2023 20:21
@snakefoot snakefoot added this to the 5.2.1 milestone Jun 14, 2023
@snakefoot
snakefoot enabled auto-merge (squash) June 14, 2023 20:32
@snakefoot
snakefoot force-pushed the assembly-prefix-not-skip branch 3 times, most recently from a8447c5 to 0fbdd33 Compare June 14, 2023 20:48
@snakefoot
snakefoot force-pushed the assembly-prefix-not-skip branch from 0fbdd33 to 60ec8e9 Compare June 14, 2023 20:48
@sonarqubecloud

Copy link
Copy Markdown

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

75.0% 75.0% Coverage
0.0% 0.0% Duplication

@snakefoot
snakefoot merged commit ae3f036 into NLog:dev Jun 14, 2023
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.

Extension prefix for assembly stopped working with NLog v5.2 - Type-alias is unknown

1 participant

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