From 990f4993173f22a5d59a5d43ec78a1e7fc31438c Mon Sep 17 00:00:00 2001 From: Pavol Loffay Date: Tue, 15 Aug 2017 07:57:30 +0200 Subject: [PATCH] fix trivial comments --- cmd/collector/app/zipkin/http_handler.go | 2 +- cmd/collector/app/zipkin/zipkin_json.go | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/cmd/collector/app/zipkin/http_handler.go b/cmd/collector/app/zipkin/http_handler.go index b07771f53af..c5de8203489 100644 --- a/cmd/collector/app/zipkin/http_handler.go +++ b/cmd/collector/app/zipkin/http_handler.go @@ -90,7 +90,7 @@ func (aH *APIHandler) saveSpans(w http.ResponseWriter, r *http.Request) { return } } else { - http.Error(w, "Only Content-Type:application/x-thrift is supported at the moment", http.StatusBadRequest) + http.Error(w, "Not supported Content-Type", http.StatusBadRequest) } if len(tSpans) > 0 { diff --git a/cmd/collector/app/zipkin/zipkin_json.go b/cmd/collector/app/zipkin/zipkin_json.go index 5f8e1ed19ba..d3f2bdf775a 100644 --- a/cmd/collector/app/zipkin/zipkin_json.go +++ b/cmd/collector/app/zipkin/zipkin_json.go @@ -163,6 +163,7 @@ func endpointToThrift(endp endpoint) (*zipkincore.Endpoint, error) { return &zipkincore.Endpoint{ ServiceName: endp.ServiceName, Port: endp.Port, + // TODO update zipkin.thrift to include ipv6 Ipv4: ipv4, }, nil }