Updates to latest brave, supporting route-based span names

[codefromthecrypt]

this tests the api design here openzipkin/brave#602

[hyleung]

I guess once this stuff is moved into the adapter, we can deprecate or remove kludgy stuff from SpanNameProvider 😁

https://github.com/hyleung/ratpack-zipkin/blob/master/src/main/java/ratpack/zipkin/SpanNameProvider.java#L11-L25

[codefromthecrypt]

ok did the minimum here. We can decide whether or not to remove the span namer independently

[codecov]

Codecov Report

Merging #58 into master will increase coverage by 0.28%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master      #58      +/-   ##
============================================
+ Coverage     81.85%   82.14%   +0.28%     
- Complexity       41       46       +5     
============================================
  Files             7        7              
  Lines           248      252       +4     
  Branches          8       11       +3     
============================================
+ Hits            203      207       +4     
  Misses           35       35              
  Partials         10       10

Impacted Files	Coverage Δ	Complexity Δ
...tpack/zipkin/internal/RatpackHttpServerParser.java	80% <ø> (ø)	5 <0> (ø)	⬇️
...ava/ratpack/zipkin/internal/ServerHttpAdapter.java	100% <100%> (ø)	13 <6> (+5)	⬆️
.../main/java/ratpack/zipkin/ServerTracingModule.java	85.71% <100%> (-0.21%)	6 <0> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c206995...316bf15. Read the comment docs.

[codefromthecrypt]

going to bump for a patch release in a bit

[codefromthecrypt]

ok ready to go. tested with the example app

[hyleung]

Thanks!