Render a URL that is a URL

[mattpodwysocki]

Found by running the thing. The Search Box API takes free text now, so this is an ordinary call:

mapbox search forward --q "Dog friendly coffee shops near me" --proximity -77.0336,38.8996

It works. The URL printed beside it did not:

[debug] GET …/forward?access_token=<redacted>&q=Dog friendly coffee shops near me&limit=1&…

$ curl "<that URL>"
HTTP 000          ← curl makes no request at all

A line whose whole purpose is "here's what was sent, go try it" couldn't be tried — and free-text queries are exactly the case that breaks it.

Only the printout was wrong

redacted_url concatenated name=value pairs raw. The request itself was never affected: reqwest encodes what it sends, which is why the call succeeded while the line beside it was unusable.

Two renderings share that function — --debug, and a text-mode --dry-run:

Dry run — nothing was sent.
POST https://api.mapbox.com/styles/v1/me?access_token=<redacted>

A JSON dry run keeps url and query as separate fields, so it never had the problem.

A value could also invent a parameter

This is the half that's worse than ugly. A value containing & split into another name=value pair, so the line claimed the request carried something it didn't, and anyone pasting it sent a different request than the one being debugged. = and + are the same class. That's the second test:

q = "coffee&limit=99&access_token=sk.theirs"
→ parses to exactly one parameter, value intact

Choices worth stating

%20, not form-urlencoding's +. Both decode to a space wherever the query is read as a form, but only %20 means a space everywhere else, and a + in a URL someone is reading is a character they have to stop and think about.

, : / @ stay literal. They're legal in a query and they're what keeps proximity=-77.0336,38.8996 readable instead of -77.0336%2C38.8996.

No new dependency. reqwest::Url was already used in this file, and the encoder is nine lines with the kept set spelled out.

Verified end to end

The printed URL, with a real token substituted for the placeholder:

HTTP 200
features: 2 | maman, petite maman

Same two features the CLI itself printed. Both new tests confirmed to fail with the encoding reverted.

599 tests, fmt and clippy clean.