Fix jaeger receiver to honor TLS config

[dstdfx]

Description:
Add missing TLSConfig into the jaeger receiver's configuration.
Create TLS listener if TLS config is passed.

Link to tracking Issue: #2850

[codecov]

Codecov Report

Merging #2866 (8d21bb0) into main (d93c9d2) will decrease coverage by 0.00%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main    #2866      +/-   ##
==========================================
- Coverage   91.61%   91.60%   -0.01%     
==========================================
  Files         312      312              
  Lines       15437    15434       -3     
==========================================
- Hits        14143    14139       -4     
- Misses        884      885       +1     
  Partials      410      410              

Impacted Files	Coverage Δ
receiver/jaegerreceiver/trace_receiver.go	73.50% <33.33%> (-1.01%)	⬇️
receiver/jaegerreceiver/factory.go	95.69% <100.00%> (+0.04%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d93c9d2...8d21bb0. Read the comment docs.

[keitwb]

The httpconfig.HTTPServerConfig used has a ToServer and ToListener method that ensures all the config is used: https://github.com/open-telemetry/opentelemetry-collector/blob/main/config/confighttp/confighttp.go#L158.

Is it possible to fit that into this receiver?

[dstdfx]

The httpconfig.HTTPServerConfig used has a ToServer and ToListener method that ensures all the config is used: https://github.com/open-telemetry/opentelemetry-collector/blob/main/config/confighttp/confighttp.go#L158.

Is it possible to fit that into this receiver?

Looks doable, just need to make sure httpconfig.HTTPServerConfig.Endpoint has a valid port as we do in the helper:
https://github.com/open-telemetry/opentelemetry-collector/blob/main/receiver/jaegerreceiver/factory.go#L242

[dstdfx]

@keitwb I think it also makes sense to use configgrpc.GRPCServerSettings for GRPC collector and use available methods for building net.Listener and server options but it's a matter of another PR, what do you think?

[keitwb]

@keitwb I think it also makes sense to use configgrpc.GRPCServerSettings for GRPC collector and use available methods for building net.Listener and server options but it's a matter of another PR, what do you think?

Yeah I agree, a different PR.

[keitwb]

@tigrannajaryan can you take a look?

[jpkrohling]

The code itself looks good to me, but haven't there been any changes to the docs?

[dstdfx]

The code itself looks good to me, but haven't there been any changes to the docs?

@jpkrohling As I see TLS configuration is already described here:
https://github.com/open-telemetry/opentelemetry-collector/blob/main/receiver/jaegerreceiver/README.md#advanced-configuration