Handle distributed context headers in custom propagator

[greenEkatherine]

Resolves #1497

Added ReverseProxyPropagator that handles headers removal. Default inner implementation removes distributed context headers. It can be changed by overriding Propagator property in ForwarderHttpClientFactory which is used to set up handler.ActivityHeadersPropagator.

[MihaZupan]

We should update the direct-forwarding docs to include changing this property on the SocketsHttpHandler as well:

reverse-proxy/docs/docfx/articles/direct-forwarding.md

Lines 51 to 57 in 96ed54f

	var httpClient = new HttpMessageInvoker(new SocketsHttpHandler()
	{
	UseProxy = false,
	AllowAutoRedirect = false,
	AutomaticDecompression = DecompressionMethods.None,
	UseCookies = false
	});

As for the breaking-change: anyone using direct forwarding will have to set the ActivityHeadersPropagator property to match existing behavior.

[greenEkatherine]

We should update the direct-forwarding docs to include changing this property on the SocketsHttpHandler as well:

reverse-proxy/docs/docfx/articles/direct-forwarding.md

Lines 51 to 57 in 96ed54f

	var httpClient = new HttpMessageInvoker(new SocketsHttpHandler()
	{
	UseProxy = false,
	AllowAutoRedirect = false,
	AutomaticDecompression = DecompressionMethods.None,
	UseCookies = false
	});

I noticed that example there doesn't match with Direct.Sample and not only with this new property. Should I update Direct.Sample as well?

[MihaZupan]

Yes please.

We should also consider updating https://microsoft.github.io/reverse-proxy/articles/header-guidelines.html#traceparent-request-id-tracestate-baggage-correlation-context to mention you can turn header removal off by calling

.ConfigureHttpHandler((_, handler) => handler.Propagator = null)

(might not be the exact syntax) in Startup

[MihaZupan]

reverse-proxy/samples/ReverseProxy.Direct.Sample/Startup.cs

Lines 36 to 42 in 96ed54f

	var httpClient = new HttpMessageInvoker(new SocketsHttpHandler()
	{
	UseProxy = false,
	AllowAutoRedirect = false,
	AutomaticDecompression = DecompressionMethods.None,
	UseCookies = false
	});

should include the ActivityHeadersPropagator.

[greenEkatherine]

reverse-proxy/samples/ReverseProxy.Direct.Sample/Startup.cs

Lines 36 to 42 in 96ed54f

	var httpClient = new HttpMessageInvoker(new SocketsHttpHandler()
	{
	UseProxy = false,
	AllowAutoRedirect = false,
	AutomaticDecompression = DecompressionMethods.None,
	UseCookies = false
	});

should include the ActivityHeadersPropagator.

Oh, forgot this one, adding...