Bug 458368

Summary: Konsole URI detection drops part of URL following a colon
Product: [Applications] konsole Reporter: burneddi
Component: generalAssignee: Konsole Developer <konsole-devel>
Status: RESOLVED FIXED    
Severity: normal CC: nicola.chiapolini
Priority: NOR    
Version: 22.04.1   
Target Milestone: ---   
Platform: Fedora RPMs   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

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!