Allow dashes according to RFC in all domain parts except TLD

[Jikstra]

Fixes #63

[puzrin]

• Domain with two dashes not being detected #63 (comment) according to discussion, no options needed for correct implementation.
• Test patterns should be placed in https://github.com/markdown-it/linkify-it/masterf/test/fixtures/links.txt when possible. Also, the real domain names required, not artificial ones.

[Jikstra]

#63 (comment) according to discussion, no options needed for correct implementation.

So you want this to be the default behaviour? I'm not too familiar with markdown and it's limitiations, so i don't know when a url with multiple dashes would cause trouble.

Test patterns should be placed in https://github.com/markdown-it/linkify-it/masterf/test/fixtures/links.txt when possible. Also, the real domain names required, not artificial ones.

Thanks, will try to update the code accordingly.

[Jikstra]

@puzrin on more question, is http://a.b--c.de/ % -- disabled, because collision possible still a url which shouldn't be allowed? Because writing a regex which checks that the dashes are only in the subdomain part is maybe a lot harder than just removing the small regex part which i made optional with the opts flag.

[puzrin]

So you want this to be the default behaviour?

Yes.

I'm not too familiar with markdown and it's limitiations, so i don't know when a url with multiple dashes would cause trouble.

If you do exactly as discussed in #63, that will be ok. We already discussed the best possible behaviour. Any deviation will require to do all checks again.

[puzrin]

New relaxed rule should touch only domain names (except TLD), and should not touch path.

[Jikstra]

My fix also allows urls like https://net--lify.com which you can't register (so i guess they are invalid). There was the one not-link test which i removed, i don't think this is a problem (and if so only a collission with markdown). If this is really a problem, i'm going to try to implement a regex which only allows multiple dashes in the second or lower levels.

[puzrin]

Also please squash everything into single commit.

[puzrin]

This 2 lines have broken formatting/indent - not like other code. Please, follow conding style for other parts of file. All you had to do - just remove (?!-) and don't touch anything else.

[puzrin]

Missed other samples, discussed in #63.

[Jikstra]

hey @puzrin is anything still blocking you from merging this?

[xPaw]

Thanks @Jikstra!

Hopefully a package update can be pushed to npm too.

[Jikstra]

@xPaw no problem, the-lounge seems nice, know some people who are using it. I'm glad i could helped you :)