chore(http): remove x-opentelemetry-outgoing-request header #1547

[vmarchaud]

Fixes #1547

[codecov]

Codecov Report

Merging #1548 into master will decrease coverage by 0.12%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1548      +/-   ##
==========================================
- Coverage   93.05%   92.92%   -0.13%     
==========================================
  Files         158      156       -2     
  Lines        4882     4851      -31     
  Branches      988      978      -10     
==========================================
- Hits         4543     4508      -35     
- Misses        339      343       +4     

Impacted Files	Coverage Δ
...ages/opentelemetry-exporter-collector/src/types.ts	100.00% <ø> (ø)
...ckages/opentelemetry-exporter-jaeger/src/jaeger.ts	85.71% <ø> (-0.37%)	⬇️
...elemetry-exporter-zipkin/src/platform/node/util.ts	96.96% <ø> (-3.04%)	⬇️
...s/opentelemetry-plugin-grpc-js/src/client/utils.ts	93.25% <ø> (-0.15%)	⬇️
packages/opentelemetry-plugin-grpc-js/src/utils.ts	92.30% <ø> (-0.80%)	⬇️
packages/opentelemetry-plugin-grpc/src/utils.ts	90.90% <ø> (-0.99%)	⬇️
packages/opentelemetry-plugin-http/src/http.ts	97.81% <ø> (-0.07%)	⬇️
packages/opentelemetry-plugin-http/src/utils.ts	97.36% <ø> (-0.06%)	⬇️
...telemetry-plugin-grpc-js/src/server/patchServer.ts	90.00% <100.00%> (-3.55%)	⬇️
packages/opentelemetry-plugin-grpc/src/grpc.ts	93.71% <100.00%> (-0.60%)	⬇️
... and 3 more

[obecny]

lgtm just wondering if some of tests could be converted using the new "suppress" feature now, WDYT ?

[vmarchaud]

just wondering if some of tests could be converted using the new "suppress" feature now, WDYT ?

Well i hesitated about that but since the supress feature is only implement in the tracing sdk, i would find it weird to add tests for that in the instrumentation, but i'm open to add some if anyone find it really important.

[mwear]

good to see this being removed!