fix(ipx): fix retrieving images with query params

[matthijsch]

Problem
images containing query params cannot be retrieved in IPX because the query params are encoded

issues
#815
#944

Solution
by using ufo's normalizeURL the src is split up in host, pathname, query etc before any encoding is done. This way only the path name is encoded

before
/_ipx/w_500%26%26q_80/https://host.com/uploads/image%20file.jpg%3Fv=1%26foo=bar
after
/_ipx/w_500%26%26q_80/https://host.com/uploads/image%20file.jpg?v=1&foo=bar

Remaining problem
This solves retrieving images, what remains to be fixed is writing the image to the disk as Nitro prevent saving images with routes containing "?" since 2.6.0:
unjs/nitro#1474

[codecov-commenter]

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (cb18ed1) 89.69% compared to head (21c45bf) 89.69%.
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #997   +/-   ##
=======================================
  Coverage   89.69%   89.69%           
=======================================
  Files          44       44           
  Lines        2920     2920           
  Branches      328      328           
=======================================
  Hits         2619     2619           
  Misses        300      300           
  Partials        1        1           

Files	Coverage Δ
src/runtime/providers/ipx.ts	100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[pi0]

Thanks for this PR.

(forwarding from discord)

While this solution might work to solve encoding issues it seems not safer. Adding URLs to the ends of a URL is already not totally safe for parsers and UFO normalize util tends to avoid extra optional encodings (following the vue-router approach). By doing this change, we can potentially introduce new edge cases.

We might investigate the upstream ipx server to handle double encoding.

I will try to follow up on this soon.

[danielroe]

@pi0 Did you end up resolving upstream in ipx? I'm aware there's been a major release since this comment so I think likely.

[barayuda]

Hi everyone @pi0 @danielroe any update regarding this?
I got this error as well when do SSG