You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
URLs such as git@github.com:robinst/linkify.git are common when working with GitHub/GitLab repositories. It would be good if linkify could reliably detect the whole URL in such cases.
I'm opening this issue to make others aware as well as to remind myself to fix this when I have time.
Hm not sure about this, as it's not really a URL in the strictest sense – the full form like ssh://git@github.com/robinst/linkify.git is a URL and works with this library today.
To support this (thinking out loud), we'd need an option to recognize SSH style "URLs", and then we'd need to extend email address scanning to also handle those and be active if the option is enabled.
I'm going to leave this open to gauge further interest, but I'm not planning to work on it for now.
Ironically, the link auto-highlighting on GitHub has the same problem highlighting the full link.
Emails
I cannot find any spec for email or mailto: links showing a colon immediately after the final domain component of the final email address as being valid.
Ambiguity
I do realize that there is a possibility for ambiguity where the GitHub/GitLab username is fully numeric.
It might be a good idea to add a LinkKind::Ambiguous to handle cases that are not clear. Further, being able to specify a policy to disambiguate to one kind or other could help a lot.
Activity
robinst commentedon Jul 11, 2022
Hm not sure about this, as it's not really a URL in the strictest sense – the full form like
ssh://git@github.com/robinst/linkify.gitis a URL and works with this library today.To support this (thinking out loud), we'd need an option to recognize SSH style "URLs", and then we'd need to extend email address scanning to also handle those and be active if the option is enabled.
I'm going to leave this open to gauge further interest, but I'm not planning to work on it for now.
egrieco commentedon Jul 16, 2022
Fair points, but it doesn't sound like you are opposed to the feature.
Is there any objection if I work on this?
egrieco commentedon Sep 18, 2022
Need
The reason I think this is necessary is that both GitHub and Gitlab provide SSH urls to repos in this format by default e.g.
Ironically, the link auto-highlighting on GitHub has the same problem highlighting the full link.
Emails
I cannot find any spec for email or
mailto:links showing a colon immediately after the final domain component of the final email address as being valid.Ambiguity
I do realize that there is a possibility for ambiguity where the GitHub/GitLab username is fully numeric.
It might be a good idea to add a
LinkKind::Ambiguousto handle cases that are not clear. Further, being able to specify a policy to disambiguate to one kind or other could help a lot.