{"action":"created","issue":{"url":"https://api.github.com/repos/contao/core/issues/7984","labels_url":"https://api.github.com/repos/contao/core/issues/7984/labels{/name}","comments_url":"https://api.github.com/repos/contao/core/issues/7984/comments","events_url":"https://api.github.com/repos/contao/core/issues/7984/events","html_url":"https://github.com/contao/core/issues/7984","id":103187286,"number":7984,"title":"Minor issue in StringUtil::encodeEmail","user":{"login":"fritzmg","id":4970961,"avatar_url":"https://avatars.githubusercontent.com/u/4970961?v=3","gravatar_id":"","url":"https://api.github.com/users/fritzmg","html_url":"https://github.com/fritzmg","followers_url":"https://api.github.com/users/fritzmg/followers","following_url":"https://api.github.com/users/fritzmg/following{/other_user}","gists_url":"https://api.github.com/users/fritzmg/gists{/gist_id}","starred_url":"https://api.github.com/users/fritzmg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fritzmg/subscriptions","organizations_url":"https://api.github.com/users/fritzmg/orgs","repos_url":"https://api.github.com/users/fritzmg/repos","events_url":"https://api.github.com/users/fritzmg/events{/privacy}","received_events_url":"https://api.github.com/users/fritzmg/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/contao/core/labels/defect","name":"defect","color":"bd2c00"},{"url":"https://api.github.com/repos/contao/core/labels/up%20for%20discussion","name":"up for discussion","color":"2a982a"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/contao/core/milestones/112","html_url":"https://github.com/contao/core/milestones/3.5.3","labels_url":"https://api.github.com/repos/contao/core/milestones/112/labels","id":1226977,"number":112,"title":"3.5.3","description":"Contao 3.5.3 (bugfix release)","creator":{"login":"leofeyer","id":1192057,"avatar_url":"https://avatars.githubusercontent.com/u/1192057?v=3","gravatar_id":"","url":"https://api.github.com/users/leofeyer","html_url":"https://github.com/leofeyer","followers_url":"https://api.github.com/users/leofeyer/followers","following_url":"https://api.github.com/users/leofeyer/following{/other_user}","gists_url":"https://api.github.com/users/leofeyer/gists{/gist_id}","starred_url":"https://api.github.com/users/leofeyer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/leofeyer/subscriptions","organizations_url":"https://api.github.com/users/leofeyer/orgs","repos_url":"https://api.github.com/users/leofeyer/repos","events_url":"https://api.github.com/users/leofeyer/events{/privacy}","received_events_url":"https://api.github.com/users/leofeyer/received_events","type":"User","site_admin":false},"open_issues":7,"closed_issues":34,"state":"open","created_at":"2015-07-27T07:31:06Z","updated_at":"2015-09-08T11:47:31Z","due_on":null,"closed_at":null},"comments":4,"created_at":"2015-08-26T06:33:28Z","updated_at":"2015-09-08T12:45:54Z","closed_at":null,"body":"The last part of this regular expression\r\n```php\r\npreg_match_all('/\\w([-._\\w]*\\w)?@\\w([-._\\w]*\\w)?\\.\\w{2,6}/', $strString, $arrEmails);\r\n```\r\ni.e. `.\\w{2,6}` seems to suggest, that you wanted to only match email addresses whose TLD is between 2 and 6 characters long. This does not actually work:\r\n\r\n![regex](https://cloud.githubusercontent.com/assets/4970961/9486873/d5f5f7ec-4bcc-11e5-8b17-71f29e79cddc.png)\r\n\r\n_However_, it's good that it does not work, since there are TLDs now with more than 6 characters, but wanted to mention it nevertheless ;). (For example, if you use the insert tag `{{email::test@test.loremipsum}}` then it will still work, even though the last part of the regex supposedly only matches TLDs between 2 and 6 characters long.)\r\n"},"comment":{"url":"https://api.github.com/repos/contao/core/issues/comments/138549066","html_url":"https://github.com/contao/core/issues/7984#issuecomment-138549066","issue_url":"https://api.github.com/repos/contao/core/issues/7984","id":138549066,"user":{"login":"aschempp","id":1073273,"avatar_url":"https://avatars.githubusercontent.com/u/1073273?v=3","gravatar_id":"","url":"https://api.github.com/users/aschempp","html_url":"https://github.com/aschempp","followers_url":"https://api.github.com/users/aschempp/followers","following_url":"https://api.github.com/users/aschempp/following{/other_user}","gists_url":"https://api.github.com/users/aschempp/gists{/gist_id}","starred_url":"https://api.github.com/users/aschempp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aschempp/subscriptions","organizations_url":"https://api.github.com/users/aschempp/orgs","repos_url":"https://api.github.com/users/aschempp/repos","events_url":"https://api.github.com/users/aschempp/events{/privacy}","received_events_url":"https://api.github.com/users/aschempp/received_events","type":"User","site_admin":false},"created_at":"2015-09-08T12:45:54Z","updated_at":"2015-09-08T12:45:54Z","body":":+1: for a library in general, but be aware that stuff like `.local` will no longer work, and I'm not sure if we should add a new dependency in a PATCH version…"}}