Support for getContextPropertiesAsync about applicationinsights-owinextensions HOT 5 CLOSED

Hi,
This seems reasonable. I've been thinking about changing the way getContextProperties works anyway, so that full OwinContext is passed into it, instead of request and response separately. I hope I'll have some time to look into it over the weekend.

from applicationinsights-owinextensions.

The work is in progress, see here. Still need to update docs and changelog though, then push to nuget.

One thing about logging the POST body: be careful. Once you read the stream, you cannot rewind it for the next middleware (like WebAPI). Recently a person told me they substitute the original stream with a MemoryStream containing copy of the data. It seemed to work, but I think it is extremely hacky. This also prevents streaming of requests.

from applicationinsights-owinextensions.

Thanks for working on that on such a short delay, and thanks for the heads up. We'll only be implementing that for the QA, front end devs are building new components and sending requests to our API that are sometimes causing issues. Logging those would help us understand what's going wrong when something goes wrong.

from applicationinsights-owinextensions.

The version 0.5 containing the changes is now released on nuget

from applicationinsights-owinextensions.

Thanks for the update and sorry for not answering earlier. I took some time off.
You're right: either the AI middleware executes before the WebAPI one and reading the stream messes up things, replacing it is hacky (made iisexpress crash on my dev box) or it executes after and then you basically cannot access the data.
I guess a proper way to do it would be to implement an attribute that executes before each method of the controller and serializes all parameters to the current operation.
Closing this one. Thanks for your help

from applicationinsights-owinextensions.