OpenTracing zipKin Support. Reference - Issue #429

Co-authored-by: Kristina Fischer <Kristina.Fischer@target.com>
Co-authored-by: Michael Murphy <Michael.Murphy@target.com>
Co-authored-by: Nathan West <nathan.west@target.com>
Co-authored-by: Austin Hartzheim <Austin.Hartzheim@target.com>
Co-authored-by: Jacob Hansen <Jacob.Hansen@target.com>

main.go

@@ -196,10 +196,10 @@ func newHTTPProxy(cfg *config.Config) http.Handler {
 			}
 			return t
 		},
-		Requests: metrics.DefaultRegistry.GetTimer("requests"),
-		Noroute:  metrics.DefaultRegistry.GetCounter("notfound"),
-		Logger:   l,
-		TracerCfg:   cfg.Tracing,
+		Requests:  metrics.DefaultRegistry.GetTimer("requests"),
+		Noroute:   metrics.DefaultRegistry.GetCounter("notfound"),
+		Logger:    l,
+		TracerCfg: cfg.Tracing,
 	}
 }
 

proxy/http_proxy.go

@@ -18,9 +18,8 @@ import (
 	"github.com/fabiolb/fabio/noroute"
 	"github.com/fabiolb/fabio/proxy/gzip"
 	"github.com/fabiolb/fabio/route"
-	"github.com/fabiolb/fabio/uuid"
 	"github.com/fabiolb/fabio/trace"
-
+	"github.com/fabiolb/fabio/uuid"
 )
 
 // HTTPProxy is a dynamic reverse proxy for HTTP and HTTPS protocols.

trace/trace.go

@@ -7,8 +7,8 @@ import (
 	"strings"
 
 	opentracing "github.com/opentracing/opentracing-go"
-	zipkin "github.com/openzipkin/zipkin-go-opentracing"
 	"github.com/opentracing/opentracing-go/ext"
+	zipkin "github.com/openzipkin/zipkin-go-opentracing"
 )
 
 func InjectHeaders(span opentracing.Span, req *http.Request) {
@@ -90,4 +90,4 @@ func InitializeTracer(collectorType string, connectString string, serviceName st
 	log.Printf("\n\nTRACER: %v\n\n", tracer)
 	log.Printf("\n\nCOLLECTOR: %v\n\n", collector)
 	log.Printf("\n\nRECORDER: %v\n\n", recorder)
-}
+}

trace/trace_test.go

@@ -2,8 +2,8 @@ package trace
 
 import (
 	opentracing "github.com/opentracing/opentracing-go"
-	mocktracer "github.com/opentracing/opentracing-go/mocktracer"
 	"github.com/opentracing/opentracing-go/ext"
+	mocktracer "github.com/opentracing/opentracing-go/mocktracer"
 	"net/http"
 	"testing"
 )