fix: encode web proxy paths using url_encode

[sherwinski]

This PR changes the encoding function used on web proxy paths from CGI.escape to ERB::Util.url_encode, which is the same function used for general path encoding in this library. Because this library uses CGI.escape to encode a web proxy path, it encodes spaces as + rather than %20 (due to following the CGI/HTML form spec). However, encoding in this way results in URLs that do not resolve properly. More details can be found in this SO discussion.

See test below with active source:

client = Imgix::Client.new(domain: "sherwinski-proxy.imgix.net", include_library_param: false, secure_url_token: ...)
path = "https://sherwinski.imgix.net/ <>[]{}|\\^%.jpg"
puts client.path(path).to_url

// main branch - 403 Forbidden
https://sherwinski-proxy.imgix.net/https%3A%2F%2Fsherwinski.imgix.net%2F+%3C%3E%5B%5D%7B%7D%7C%5C%5E%25.jpg?s=35ca40e2e7b6bd208be2c4f7073f658e

// this branch - 200 OK
https://sherwinski-proxy.imgix.net/https%3A%2F%2Fsherwinski.imgix.net%2F%20%3C%3E%5B%5D%7B%7D%7C%5C%5E%25.jpg?s=2c215192a0fb2ee4c075cb7f6b57ea59

[commit-lint]

Bug Fixes

• encode web proxy paths using url_encode (9bf57ba)

Tests

• ensure spaces are properly encoded (d52755d)

Contributors

sherwinski

Commit-Lint commands

You can trigger Commit-Lint actions by commenting on this PR:

• @Commit-Lint merge patch will merge dependabot PR on "patch" versions (X.X.Y - Y change)
• @Commit-Lint merge minor will merge dependabot PR on "minor" versions (X.Y.Y - Y change)
• @Commit-Lint merge major will merge dependabot PR on "major" versions (Y.Y.Y - Y change)
• @Commit-Lint merge disable will desactivate merge dependabot PR
• @Commit-Lint review will approve dependabot PR
• @Commit-Lint stop review will stop approve dependabot PR

[ericdeansanchez]

:chefkiss: 👍