Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Improve Linkifier #2011

Merged
merged 3 commits into from
Jun 25, 2018
Merged

Improve Linkifier #2011

merged 3 commits into from
Jun 25, 2018

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented Jun 23, 2018

Update linkifyjs dependency so we get the new list of TLDs: Hypercontext/linkifyjs#213

  • allow TLD to feed back into domain, in case there is something that looks like a TLD multiple times anchored at the end of the input.
  • allow ((DOMAIN|TLD).)+ instead of just (DOMAIN.)+ in case there is something that looks like a TLD followed by something that doesn't look like TLD

Fixes element-hq/element-web#2756 and element-hq/element-web#3514

image

the remaining failures are caused by them ending in invalid TLDs

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@dbkr dbkr merged commit 6577671 into develop Jun 25, 2018
@t3chguy t3chguy deleted the t3chguy/improve_linkifier branch May 25, 2020 18:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants