-
Notifications
You must be signed in to change notification settings - Fork 165
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
${aspnet-request-ip} has not been updated to use TryGetRequest #274
Labels
Milestone
Comments
I have time to look at this, so will try to fix this |
Hi. I'm not really familiar with github.
You can instruct me of how to create the PR if you want, or I can send you
the specific line of code where it should be corrected. whichever works for
you.
thanks!
…On Thu, Apr 12, 2018 at 6:27 PM Julian Verdurmen ***@***.***> wrote:
I have time to look at this, so will try to fix this
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#274 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABf8LG8-gjcch00bEw3I4Ip05-HI33Kuks5tn3J9gaJpZM4TRn5q>
.
|
This is ASP.NET and not ASP.NET Core, isn't? |
I've made already a PR :) |
Wow you are quick. |
It's now publishing. |
It's online! https://www.nuget.org/packages/NLog.Web/4.5.3 |
this was the fastest issue solving/publishing that I had ever witnessed. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As a result, it still throws an HttpException with the message: "Request is not available in this context"
Please change it to use TryGetRequest in order to avoid this exception.
The text was updated successfully, but these errors were encountered: