mdsol / Medidata.ZipkinTracerModule

[Deprecated] Zipkin Request Tracing for .Net Apps
MIT License
53 stars 20 forks source link

IPv6 support #88

Open codefromthecrypt opened 8 years ago

codefromthecrypt commented 8 years ago

We currently support logging of only IPv4 addresses. Starting with Zipkin 1.4 endpoints can omit IPv4 (by setting Endpoint.ipv4 to 0), and optionally log Endpoint.ipv6 as the raw 16byte address. https://github.com/openzipkin/zipkin-api/blob/master/thrift/zipkinCore.thrift#L276

In json, both are the string formatting http://zipkin.io/zipkin-api

ipv4:
string The text representation of a IPv4 address associated with this endpoint. Ex. 192.168.99.100 ipv6:
string The text representation of a IPv6 address associated with this endpoint. Ex. 2001:db8::c001

jcarres-mdsol commented 8 years ago

Made it an enhancement. @bvillanueva-mdsol @lschreck-mdsol @kenyamat FYI