Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Some character sequences break URL detection #251

Open
atraczyk opened this issue Mar 6, 2024 · 2 comments
Open

Some character sequences break URL detection #251

atraczyk opened this issue Mar 6, 2024 · 2 comments

Comments

@atraczyk
Copy link

atraczyk commented Mar 6, 2024

/-/ and /+/ prevents URL detection when calling md_html.

Example broken link: https://codereview.qt-project.org/c/qt/qtwayland/+/545836

Works fine if enveloped in chevrons: <https://codereview.qt-project.org/c/qt/qtwayland/+/545836>

Tested with commit 481fbfb
@Kaffeine
Copy link

Kaffeine commented Aug 2, 2024

The same goes for underscore. E.g. https://www.youtube.com/watch?v=_tEL8bJ7yqU

Kaffeine added a commit to Kaffeine/md4c that referenced this issue Aug 2, 2024
@Kaffeine
md_analyze_permissive_autolink: Fix allowed nonalnum chars handling
46c9492 
Partially fixes mity#251.
Kaffeine added a commit to Kaffeine/md4c that referenced this issue Aug 2, 2024
@Kaffeine
md_analyze_permissive_autolink: Allow '+' in URL paths
f25d6a0 
Fixes mity#251.
@Kaffeine Kaffeine mentioned this issue Aug 2, 2024
Open
@Page-MS
Copy link

Page-MS commented Feb 13, 2025

I encountered the exact same issue.
Is it planned to be fixed ?

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Kaffeine @atraczyk @Page-MS