convert foo.com

[lsmith77]

imho it would make sense to also convert test.com type strings

1) Liip\UrlAutoConverterBundle\Tests\Extension\UrlAutoConverterTwigExtensionTest::testAutoConvertUrls with data set #1 ('Lorem ipsum dolor sit amet, <a href="http://test.com">test.com</a> consectetuer adipiscing', 'Lorem ipsum dolor sit amet, test.com consectetuer adipiscing')
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'Lorem ipsum dolor sit amet, <a href="http://test.com">test.com</a> consectetuer adipiscing'
+'Lorem ipsum dolor sit amet, test.com consectetuer adipiscing'

1) Liip\UrlAutoConverterBundle\Tests\Extension\UrlAutoConverterTwigExtensionTest::testAutoConvertUrls with data set #3 ('Lorem ipsum dolor sit amet, lala <a href="http://subtest.test.com">subtest.test.com</a> aksjdhasd.', 'Lorem ipsum dolor sit amet, lala subtest.test.com aksjdhasd.')
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'Lorem ipsum dolor sit amet, lala <a href="http://subtest.test.com">subtest.test.com</a> aksjdhasd.'
+'Lorem ipsum dolor sit amet, lala subtest.test.com aksjdhasd.'

[prionkor]

+1 on this. If not default we could turn this feature on with some flags.

[dbu]

ping

[iamcal]

feel free to send a pull request

doing this gets complicated - what TLDs do you support? do some TLDs requires paths? (else the perl script foo.pl is suddenly a URL)

[cweiske]

That list is called the "public suffix list" and can be found at https://publicsuffix.org/