Autolinking as specified by the GFM spec #1459

Closed
opened 2025-11-02 04:01:31 -06:00 by GiteaMirror · 2 comments
Owner

Originally created by @thehowl on GitHub (Jan 16, 2018).

After some more research about how GitHub handles autolinking, I wound up finding the proper GFM spec. One of the goals of #3354 is that of porting over functionality, which was previously provided by JavaScript, into Go, which is why I do not intend to include this as part of the PR, seeing as that is already quite big. However, once it is merged, it might be good to replace the linkProcessor in modules/markup/html.go with a proper processor following the GFM specification - which should hopefully solve once and for all all issues regarding autolinking not working.

This would also fix #3095.

cc/ @ypnos

Originally created by @thehowl on GitHub (Jan 16, 2018). After some more research about how GitHub handles autolinking, I wound up finding the proper [GFM](https://github.github.com/gfm/#autolinks-extension-) spec. One of the goals of #3354 is that of porting over functionality, which was previously provided by JavaScript, into Go, which is why I do not intend to include this as part of the PR, seeing as that is already quite big. However, once it is merged, it might be good to replace the `linkProcessor` in `modules/markup/html.go` with a proper processor following the GFM specification - which should hopefully solve once and for all all issues regarding autolinking not working. This would also fix #3095. cc/ @ypnos
GiteaMirror added the issue/confirmedtype/refactoring labels 2025-11-02 04:01:31 -06:00
Author
Owner

@stale[bot] commented on GitHub (Feb 8, 2019):

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale[bot] commented on GitHub (Feb 8, 2019): This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.
Author
Owner

@techknowlogick commented on GitHub (Mar 7, 2019):

Closing per merged PR

@techknowlogick commented on GitHub (Mar 7, 2019): Closing per merged PR
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#1459