Review Request 129208: Fix KUrl::isRelativeUrl to allow all RFC3986 characters in scheme.

Gerhard Gappmeier gappy1502 at gmx.net
Mon Oct 17 11:18:31 BST 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129208/
-----------------------------------------------------------

Review request for kdelibs.


Repository: kdelibs


Description
-------

RFC3986 defines the following syntax for URI schemes
scheme = alpha *( alpha | digit | "+" | "-" | "." )

The '.' was missing which leaded to the problem that
xdg-open opc.tcp://localhost didn't work.

Background:
xdg-open calls kde-open which then uses KUrl and so leaded to this
problem. Of course this bug may affect also many other locations
using KUrl::isRelativeUrl.

This fix also adds a unit test to test the fix and avoid future
regressions.


Diffs
-----

  kdecore/io/kurl.cpp d9661eab5f8df3bfbbd2900246437fb129037dba 
  kdecore/tests/kurltest.cpp 3947724bfd1d26ad69b97789205a814a8d712591 

Diff: https://git.reviewboard.kde.org/r/129208/diff/


Testing
-------


Thanks,

Gerhard Gappmeier

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20161017/48425a7e/attachment.htm>


More information about the kde-core-devel mailing list