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

Mark AspNetLayoutRendererBase as obsolete since DoAppend gives little value #846

Open
snakefoot opened this issue Aug 9, 2022 · 2 comments

Comments

@snakefoot
Copy link
Contributor

The pre-validation of whether active HttpContext is available provides very little value, but gives a performance hit.

@bakgerman
Copy link
Contributor

Completed this in a branch until ready for merge

@snakefoot
Copy link
Contributor Author

snakefoot commented Aug 21, 2023

Thank you for making the hard work. But notice that NLog v6 is still far into the future (maybe a year or more), so your branches might grow stale (reason why issues was not marked as "up-for-graps")

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

No branches or pull requests

2 participants