Don't check DNS on startup for UdpSender

jaegertracing / jaeger-client-csharp

🛑 This library is DEPRECATED!

https://jaegertracing.io/

Apache License 2.0

304 stars 67 forks source link

Don't check DNS on startup for UdpSender #168

Closed Falco20019 closed 4 years ago

Falco20019 commented 4 years ago

Which problem is this PR solving?

Fixes #153

Short description of the changes

Checking is postponed from creation time to reporting time
WIP until #167 is merged and code is checked

Falco20019 commented 4 years ago

@yurishkuro We now have changed the behavior, so that we are fine with resolving the host only on sending, not on creation. I tested it locally with crossdock by also removing the depends-on which fails without this change.

@saschaholesch Please have a look.