739
100%
master: 100%

Ran 09 Jun 2021 05:34PM UTC

Jobs 3

Files 32

Run time 46s

Badge

Embed ▾

pending completion

Build # 739

Build Type

push

travis-ci

Committed by

Sinjo

Commit Message

Fix URI escaping of spaces in push.rb

When `URI.escape` was deprecated, we switched to using `CGI.escape` to
construct the request path in push.rb.

The problem with that is that it mishandles spaces, encoding them as `+`
rather than `%20`. This leads to literal `+` characters in metric
labels, which is a bug.

This commit uses `ERB::Util.url_encode` instead, which gives us the
behaviour we want.

It's not the part of the standard library I expected to use for this,
but it's the only one that seems to have the behaviour we want and that
hasn't disappeared in Ruby 3.0. `WEBrick::HTTPUtils.escape` seems
similar on the surface, but doesn't encode `&`. Presumably it's intended
as a best-effort way to encode an entire URL, and has to assume that any
`&` character is a separator between query params.

Run Details

1634 of 1634 relevant lines covered (100.0%)

47.17 hits per line

Jobs

ID	Job ID	Ran	Coverage
1	739.1 (2.5.8)	09 Jun 2021 05:34PM UTC	100.0	Travis Job 739.1
2	739.2 (2.6.6)	09 Jun 2021 05:34PM UTC	100.0	Travis Job 739.2
3	739.3 (2.7.2)	09 Jun 2021 05:34PM UTC	100.0	Travis Job 739.3

prometheus / client_ruby / 739
100%
master: 100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 739

prometheus / client_ruby / 739 100% master: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 739

prometheus / client_ruby / 739
100%
master: 100%

README BADGES
x