Developers tools incorrectly highlight and open url containing matching parentheses
Reported by
i.tsur...@gmail.com,
Jan 4 2017
|
|||||
Issue description
UserAgent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/55.0.2883.87 Safari/537.36
Steps to reproduce the problem:
1. console.log("http://example.com?_g=()");
2. Point over the url
3. Click on it
What is the expected behavior?
The whole url should be highlighted
What went wrong?
Closing parentheses are ignored, the url is treated as "http://example.com?_g=("
Did this work before? N/A
Chrome version: 55.0.2883.87 Channel: stable
OS Version: 6.1 (Windows 7, Windows Server 2008 R2)
Flash Version: Shockwave Flash 24.0 r0
,
Jan 6 2017
Able to reproduce the issue on Win 10,Mac 10.12.2 and Ubuntu 14.04 using 55.0.2883.87/95 and canary 57.0.2973.0. This is a non-regression issue since 35.0.1916.0. Note: Its working fine on FireFox 50.0.2.
,
Jan 10 2017
,
Jan 10 2017
Thanks for the report. We came across bugs similar to this before, see crbug.com/535463 . It's unlikely that a single regex will reliably capture all valid URI schemes, which is why attempts to update our linkifier have missed cases. It's not uncommon to see strings like "(www.chromium.org)" in console output, which need to be linkified as "www.chromium.org". If we aggressively include parenthesis in a url, the string "(www.chromium.org/?=)" would be linkified as "www.chromium.org/?=)". However, this is not correct if I was using the pair of parenthesis as a group. We've tried lookbehind assertions before, but if you have other ideas on things to try, please let us know.
,
Mar 17 2017
Issue 699291 has been merged into this issue.
,
Sep 19 2017
This has been approached before, but we couldn't find a great performant solution. Marking as wontfix. |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by nyerramilli@chromium.org
, Jan 5 2017