[pkg-go] Bug#926380: golang-github-puerkitobio-purell: FTBFS (failing tests)

Andreas Henriksson andreas at fatal.se
Sun Apr 7 15:05:32 BST 2019

Greetings from the Gothenburg BSP.

This is the output I get when reproducing this issue:


--- FAIL: TestEncodeNecessaryEscapesAll (0.00s)
    purell_test.go:761: Got error parse http://host/

� net/url: invalid control character in URL
exit status 1
FAIL	_/tmp/golang-github-puerkitobio-purell-1.1.0	0.003s


net/url Parse now explicitly check that you don't pass in
"invalid" (non-encoded) urls and rejects them:

The NormalizeUrlString helper function starts out by passing the url
string to url.Parse:

The TestDecodeUnnecessaryEscapesAll function creates an url with control
characters and passes it to NormalizeURLString to try to get it

I'd say the design of the NormalizeUrlString function is broken and thus
it's not obvious to me how to fix it. I'd say that if you have a
non-normalized string you want encoded, you should pass it in as
something that doesn't need to be parsed. ie. use NormalizeUrl helper
function, which takes an Url type.

Removing the NormalizeUrlString function however would be an API/ABI
break as it's publicly exported (and used by Hugo and Kubernetes)...

Maybe open-coding some homebrew url parsing to fall back on could be
done to keep the function around..... Sounds bad to think you know
better than net/url how to parse an url though.

I'm attaching a patch which "fixes" this issue, but really it's most likely
a stupid and wrong things to do to solve this. It's just designed to
make the testsuite pass. As mentioned, I think the NormalizeURLString
function is incorrectly designed and there's no way to fix it (so should
be deprecated in favour of NormalizeURL).
Thus intentionally *not* tagging patch, as this is rather a "proof of

Andreas Henriksson
-------------- next part --------------
A non-text attachment was scrubbed...
Name: purell-url.diff
Type: text/x-diff
Size: 1702 bytes
Desc: not available
URL: <http://alioth-lists.debian.net/pipermail/pkg-go-maintainers/attachments/20190407/e78b2957/attachment.diff>

More information about the Pkg-go-maintainers mailing list