https://bugs.kde.org/show_bug.cgi?id=386406

Ahmad Samir <a.samir...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://invent.kde.org/fram
                   |                            |eworks/kio/commit/698f6d2ff
                   |                            |4193a3c6dcde900f4287dab0808
                   |                            |7399
             Status|REPORTED                    |RESOLVED
         Resolution|---                         |FIXED
   Version Fixed In|                            |5.71.0

--- Comment #1 from Ahmad Samir <a.samir...@gmail.com> ---
Git commit 698f6d2ff4193a3c6dcde900f4287dab08087399 by Ahmad Samir.
Committed on 01/06/2020 at 15:20.
Pushed by dfaure into branch 'master'.

[kio_http] Parse a FullyEncoded QUrl path with TolerantMode

When using url.setPath(otherUrl.path(FullyEncoded)), we have to specify the
TolerantMode parsing mode (setPath uses DecodedMode by default), otherwise
we end up with a doubly encoded url:

otherUrl.path(FullyEncoded) would represent a space as %20 then
url.setPath(otherUrl.path(FullyEncoded), DecodedMode) would represent
the '%' character with '%25' so the path ends up with '%2520'.

AFAIU, either we use:
url.setPath(otherUrl.path(FullyDecoded), DecodedMode) OR
url.setPath(otherUrl.path(FullyEncoded), TolerantMode)

url.path() and url.setPath use FullyDecoded and DecodedMode by default,
respectively.

Thanks to Andreas Schwab for figuring it out in the bug report.
FIXED-IN: 5.71.0

M  +1    -1    src/ioslaves/http/http.cpp

https://invent.kde.org/frameworks/kio/commit/698f6d2ff4193a3c6dcde900f4287dab08087399

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to