Michael Morozov

Results 8 comments of Michael Morozov

This bit should fix https://github.com/labstack/echo-contrib/pull/71

Good catch @mgutijae. The issue is easy to reproduce. Just create traced request for HTTP POST and try to send it via http.Client. You will propagate only tracing headers but...

> Could someone wit actual jaeger instance confirm that it works and I'll merge this PR. Sure. I can check this.

@aldas I did a small concept to prove the fix is working: https://github.com/m1x0n/echo-jaeger/blob/master/srv.go Results: ```bash ~ curl -i http://localhost:1337/broken HTTP/1.1 400 Bad Request Content-Type: text/plain; charset=UTF-8 Date: Fri, 01 Jul...

@aldas The issue is still relevant. Are there any other checks we need to add to get fix merged?

Great! Thank you.

Fixed in https://github.com/labstack/echo-contrib/releases/tag/v0.13.1

Hi, @TechOverflow. Since the `property` could be on some nested level it should be possible to resolve its name from `pointer` value. It looks like there is a bug in...