markdown-it / linkify-it

Links recognition library with full unicode support
http://markdown-it.github.io/linkify-it/
MIT License
655 stars 63 forks source link

URLs with period and parantheses stop before closing parentheses #94

Closed JeremyMahieu closed 2 years ago

JeremyMahieu commented 3 years ago

Links such as

For example real world links can be file shares that replace spaces in filenames with periods. (note: I have no affiliation with the link or the content, just found on google when searching for "Index of"

nmilev-scribe commented 2 years ago

Having the same issue. How to reproduce: paste the url below into the lounge demo:

https://mixpanel.com/project/2224040/view/2771788/app/insights#report/28200585/kp-is/~(columnWidths~(bar~())~description~'~displayOptions~(chartType~'line~plotStyle~'standard~analysis~'linear~value~'absolute)~sorting~(bar~(sortBy~'column~colSortAttrs~(~(sortBy~'value~sortOrder~'desc)))~line~(sortBy~'value~sortOrder~'desc~valueField~'averageValue~colSortAttrs~(~(sortBy~'value~sortOrder~'desc~valueField~'averageValue)))~table~(sortBy~'column~colSortAttrs~(~(sortBy~'label~sortOrder~'asc)))~insights-metric~(sortBy~'value~sortOrder~'desc~valueField~'totalValue~colSortAttrs~(~(sortBy~'value~sortOrder~'desc~valueField~'totalValue)))~pie~(sortBy~'value~sortOrder~'desc~valueField~'totalValue~colSortAttrs~(~(sortBy~'value~sortOrder~'desc~valueField~'totalValue))))~timeComparison~null~querySamplingEnabled~false~dashboard~(id~2868932~title~'NTRS*20*281000*29~creator_name~'Caila*20MacDonald~can_update_basic~true~can_share~true~allow_staff_override~false~is_superadmin~false)~title~'KPIs~sections~(show~(~(datase
rlidwka commented 2 years ago

Fixed examples like foo.(bar), foo;[bar] and similar (with punctuation followed by some sort of parenthesis).

For deeply nested parens we don't have a solution, see https://github.com/markdown-it/linkify-it/issues/23

nmilev-scribe commented 2 years ago

@rlidwka Which version contains this fix?