Links containing underscores don't get oneboxed

I noticed that links containing underscores (_) don’t get oneboxed correctly under certain conditions, because the underscores get interpreted as formatting directives.

Consider this link:

https://github.com/arve0/fijibin/blob/a3d2e983cb9ff2bcbb56a800084bc3b35cb9292f/fijibin/__init__.py#L23

Trying to onebox:

https://github.com/arve0/fijibin/blob/a3d2e983cb9ff2bcbb56a800084bc3b35cb9292f/fijibin/init.py#L23

This can be worked around using %5F to replace the first underscore:

https://github.com/arve0/fijibin/blob/a3d2e983cb9ff2bcbb56a800084bc3b35cb9292f/fijibin/%5F_init__.py#L23

https://github.com/arve0/fijibin/blob/a3d2e983cb9ff2bcbb56a800084bc3b35cb9292f/fijibin/__init__.py#L23

simply put brackets around it like so

<https://github.com/arve0/fijibin/blob/a3d2e983cb9ff2bcbb56a800084bc3b35cb9292f/fijibin/__init__.py#L23>

Not a bug, auto linking cannot be expected to work in all scenarios. See http://commonmark.org/help for the markdown spec on this

7 Likes

That doesn’t get oneboxed, either (the percent encoding method seems like the right way to work around this).

2 Likes

I agree with @notriddle: adding the brackets (<>) will keep the link intact, but actually prevents oneboxing in my trials.

1 Like

I am half open to allowing oneboxing (but not mini oneboxing) it is quite an edge case though. The bug is in linkify: https://github.com/markdown-it/linkify-it so may be worth mentioning in an issue there.

5 Likes