Bug 458368 - Konsole URI detection drops part of URL following a colon
Summary: Konsole URI detection drops part of URL following a colon
Status: RESOLVED FIXED
Alias: None
Product: konsole
Classification: Applications
Component: general (show other bugs)
Version: 22.04.1
Platform: Fedora RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: Konsole Developer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-08-27 08:43 UTC by burneddi
Modified: 2022-10-15 12:38 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description burneddi 2022-08-27 08:43:21 UTC
SUMMARY

Some URIs contain the : symbol as a necessary part of the URI. For example, links to Twitter's image CDN, such as https://pbs.twimg.com/media/FbCHp9lWQAA7uUN.jpg:orig (where the :orig part gives you a higher-resolution image) and links that include a port number at the end, such as http://localhost:8080/index.php

The URI regex changed recently and no longer works with links such as these.


STEPS TO REPRODUCE
1. Type "http://localhost:8080/index.php" into Konsole
2. Move mouse over the URI

OBSERVED RESULT

Only "http://localhost" is clickable

EXPECTED RESULT

"http://localhost:8080/index.php" should be clickable

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Fedora 36
(available in About System)
KDE Plasma Version: 5.25.4
KDE Frameworks Version: 5.96.0
Qt Version: 5.15.5

ADDITIONAL INFORMATION
Comment 1 burneddi 2022-08-27 20:04:48 UTC
Noticed that commas also do not work. For example https://images.cdn.yle.fi/image/upload/w_1200,h_800,ar_1.4992504,dpr_1,c_fill/q_auto:eco,f_auto,fl_lossy/39-83346060f597a4c07f1 ends the link after "w_1200", breaking it.
Comment 2 burneddi 2022-10-13 17:41:15 UTC
The regexp also does not include closing parenthesis in URLs that have them. For example, the URL https://en.wikipedia.org/wiki/Turnstile_(symbol) does not work.
Comment 3 burneddi 2022-10-15 12:38:02 UTC
I updated to 22.08.1 and all examples mentioned here seem to now be picked up correctly by the URI regex. Hooray!