{"action":"closed","issue":{"url":"https://api.github.com/repos/lukeed/matchit/issues/4","repository_url":"https://api.github.com/repos/lukeed/matchit","labels_url":"https://api.github.com/repos/lukeed/matchit/issues/4/labels{/name}","comments_url":"https://api.github.com/repos/lukeed/matchit/issues/4/comments","events_url":"https://api.github.com/repos/lukeed/matchit/issues/4/events","html_url":"https://github.com/lukeed/matchit/issues/4","id":291426240,"number":4,"title":"url parse length","user":{"login":"crazy4groovy","id":1110812,"avatar_url":"https://avatars0.githubusercontent.com/u/1110812?v=4","gravatar_id":"","url":"https://api.github.com/users/crazy4groovy","html_url":"https://github.com/crazy4groovy","followers_url":"https://api.github.com/users/crazy4groovy/followers","following_url":"https://api.github.com/users/crazy4groovy/following{/other_user}","gists_url":"https://api.github.com/users/crazy4groovy/gists{/gist_id}","starred_url":"https://api.github.com/users/crazy4groovy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/crazy4groovy/subscriptions","organizations_url":"https://api.github.com/users/crazy4groovy/orgs","repos_url":"https://api.github.com/users/crazy4groovy/repos","events_url":"https://api.github.com/users/crazy4groovy/events{/privacy}","received_events_url":"https://api.github.com/users/crazy4groovy/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2018-01-25T02:07:31Z","updated_at":"2018-05-04T13:36:07Z","closed_at":"2018-05-04T13:36:07Z","author_association":"NONE","body":"Hi, neat lib!\r\n\r\nA lot of libraries have a DoS vulnerability due to parsing times for very long inputs (ie url strings ~100K long).\r\n\r\nI'm looking at https://github.com/lukeed/matchit/blob/master/lib/index.js#L19\r\nand wondering if there's an issue here? Recursion stack overflow? Might be worth at least a benchmark test.\r\n\r\n(I suppose the easiest solution is to cap the length at something reasonable like 1k)"}}
{"actor":{"display_login":"crazy4groovy"}}