Make Key lower case by using HTTPHeaders

grpc-ecosystem / grpc-opentracing

OpenTracing is a set of consistent, expressive, vendor-neutral APIs for distributed tracing and context propagation

BSD 3-Clause "New" or "Revised" License

472 stars 98 forks source link

Make Key lower case by using HTTPHeaders #6

Closed zirkome closed 8 years ago

zirkome commented 8 years ago

This changes the key mapper from TextMap to HTTPHeaders.

I integrated the work of @bajb in the commits to give him credit of the finding.

I made this PR so it can be merged quicker as I need this fix in order to make it work with Zipkin.

shouichi commented 8 years ago

@bensigelman Could you take a look at the PR? Without this change, we'll get rpc error: code = 13 desc = stream terminated by RST_STREAM with error code: 1 (I'm using zipkin).