Upgrading Application Insights breaks service correlation about applicationinsights-servicefabric HOT 7 OPEN

@lindgrend, most likely it's because the latest AppInsights sdk switched to W3C correlation header, which is not compatible with the correlation protocol in this SDK.
And there is currently no plan on supporting the new protocol for this SDK, so unfortunately, you'll have to choose between one of them.

from applicationinsights-servicefabric.

Am I understanding it correctly that it could be solved by upgrading Service fabric and would that include going to the next major release then?

from applicationinsights-servicefabric.

It's correct it can be solved by implementing the new correlation protocol.
However, this repo is currently in maintenance mode only for bug fixes, there is no plan for next major release. :(

from applicationinsights-servicefabric.

@yantang-msft I hop on this issue based on a issue I encounter (https://stackoverflow.com/questions/66011506/applicationinsights-end-2-end-breaks-on-servicefabric-apigateway) which looks like this issue.

Could you clarify a thing
==> However, this repo is currently in maintenance mode only for bug fixes, there is no plan for next major release. :( <==

Does this mean there are no updates coming for ApplicationInsights for ServiceFabric? Because the documentation states that "The correlation HTTP protocol, also called Request-Id, is being deprecated".

What's the best way forward? Forcing it to the "old" correlation protocol?

Hope my question is clear, maybe I misunderstood it a bit ;-)

from applicationinsights-servicefabric.

@DHoefakker I at one time saw there is a way to force using Request-Id in the doc, but last time I checked the doc again, I don't see it anymore. @cijothomas do you know how to enforce Request-Id protocol for the .net SDK?

from applicationinsights-servicefabric.

@yantang-msft I believe ApplicationInsights still does request-id propagation along with traceparent. It would respect the ActivityID format set by the user. So one can set ActivityIDformat to legacy one like below. (somewhere in app startup)

Activity.DefaultIdFormat = ActivityIdFormat.Hierrachical;
Activity.ForceDefaultIdFormat = true;

If this doesn't work, please open an issue in the applicaiton insights repo.

from applicationinsights-servicefabric.

@cijothomas @yantang-msft Yes, setting the ActivityIDFormat did the trick 🥳

Beside that i am wondering does this mean there are no updates coming for ApplicationInsights for ServiceFabric? Or is it related to .net? Are you now focussing on .net core / .net 5, and maintaining only the .net < 4.x version ?

from applicationinsights-servicefabric.