{"action":"created","issue":{"url":"https://api.github.com/repos/symfony/symfony/issues/13522","labels_url":"https://api.github.com/repos/symfony/symfony/issues/13522/labels{/name}","comments_url":"https://api.github.com/repos/symfony/symfony/issues/13522/comments","events_url":"https://api.github.com/repos/symfony/symfony/issues/13522/events","html_url":"https://github.com/symfony/symfony/pull/13522","id":55446735,"number":13522,"title":"[2.7] [HttpFoundation] [Request] Added check for valid host size limit","user":{"login":"phansys","id":1231441,"avatar_url":"https://avatars.githubusercontent.com/u/1231441?v=3","gravatar_id":"","url":"https://api.github.com/users/phansys","html_url":"https://github.com/phansys","followers_url":"https://api.github.com/users/phansys/followers","following_url":"https://api.github.com/users/phansys/following{/other_user}","gists_url":"https://api.github.com/users/phansys/gists{/gist_id}","starred_url":"https://api.github.com/users/phansys/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/phansys/subscriptions","organizations_url":"https://api.github.com/users/phansys/orgs","repos_url":"https://api.github.com/users/phansys/repos","events_url":"https://api.github.com/users/phansys/events{/privacy}","received_events_url":"https://api.github.com/users/phansys/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":16,"created_at":"2015-01-26T04:53:09Z","updated_at":"2015-01-27T16:09:19Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/symfony/symfony/pulls/13522","html_url":"https://github.com/symfony/symfony/pull/13522","diff_url":"https://github.com/symfony/symfony/pull/13522.diff","patch_url":"https://github.com/symfony/symfony/pull/13522.patch"},"body":"| Q             | A\r\n| ------------- | ---\r\n| Bug fix?      | no\r\n| New feature?  | yes\r\n| BC breaks?    | kinda\r\n| Deprecations? | no\r\n| Tests pass?   | yes\r\n| Fixed tickets | #12349\r\n| License       | MIT\r\n| Doc PR        | none\r\n\r\nAdded check for valid host size limit according to RFC 1035 - 2.3.4 (ticket #12349):\r\nhttp://www.ietf.org/rfc/rfc1035.txt\r\n* 63 bytes limit for each domain label\r\n* 255 bytes limit for whole domain name\r\n\r\nTODO:\r\n- [ ] Replace preg_match() by preg_replace()."},"comment":{"url":"https://api.github.com/repos/symfony/symfony/issues/comments/71674098","html_url":"https://github.com/symfony/symfony/pull/13522#issuecomment-71674098","issue_url":"https://api.github.com/repos/symfony/symfony/issues/13522","id":71674098,"user":{"login":"phansys","id":1231441,"avatar_url":"https://avatars.githubusercontent.com/u/1231441?v=3","gravatar_id":"","url":"https://api.github.com/users/phansys","html_url":"https://github.com/phansys","followers_url":"https://api.github.com/users/phansys/followers","following_url":"https://api.github.com/users/phansys/following{/other_user}","gists_url":"https://api.github.com/users/phansys/gists{/gist_id}","starred_url":"https://api.github.com/users/phansys/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/phansys/subscriptions","organizations_url":"https://api.github.com/users/phansys/orgs","repos_url":"https://api.github.com/users/phansys/repos","events_url":"https://api.github.com/users/phansys/events{/privacy}","received_events_url":"https://api.github.com/users/phansys/received_events","type":"User","site_admin":false},"created_at":"2015-01-27T16:09:19Z","updated_at":"2015-01-27T16:09:19Z","body":"Thanks @dunglas, it seems to be the cleaner solution.\r\nI just rebased the PR to my first commit and I added tests for IDN hostnames (raw and encoded with Punycode).\r\nDepending in if gets merged or not, #12349 can be closed as \"resolved\" or \"won't fix\"."}}