{"action":"created","issue":{"url":"https://api.github.com/repos/rails/rails/issues/20797","labels_url":"https://api.github.com/repos/rails/rails/issues/20797/labels{/name}","comments_url":"https://api.github.com/repos/rails/rails/issues/20797/comments","events_url":"https://api.github.com/repos/rails/rails/issues/20797/events","html_url":"https://github.com/rails/rails/pull/20797","id":93597524,"number":20797,"title":"Prevent ActionController::Parameters in url_for","user":{"login":"byroot","id":44640,"avatar_url":"https://avatars.githubusercontent.com/u/44640?v=3","gravatar_id":"","url":"https://api.github.com/users/byroot","html_url":"https://github.com/byroot","followers_url":"https://api.github.com/users/byroot/followers","following_url":"https://api.github.com/users/byroot/following{/other_user}","gists_url":"https://api.github.com/users/byroot/gists{/gist_id}","starred_url":"https://api.github.com/users/byroot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/byroot/subscriptions","organizations_url":"https://api.github.com/users/byroot/orgs","repos_url":"https://api.github.com/users/byroot/repos","events_url":"https://api.github.com/users/byroot/events{/privacy}","received_events_url":"https://api.github.com/users/byroot/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"pixeltrix","id":6321,"avatar_url":"https://avatars.githubusercontent.com/u/6321?v=3","gravatar_id":"","url":"https://api.github.com/users/pixeltrix","html_url":"https://github.com/pixeltrix","followers_url":"https://api.github.com/users/pixeltrix/followers","following_url":"https://api.github.com/users/pixeltrix/following{/other_user}","gists_url":"https://api.github.com/users/pixeltrix/gists{/gist_id}","starred_url":"https://api.github.com/users/pixeltrix/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pixeltrix/subscriptions","organizations_url":"https://api.github.com/users/pixeltrix/orgs","repos_url":"https://api.github.com/users/pixeltrix/repos","events_url":"https://api.github.com/users/pixeltrix/events{/privacy}","received_events_url":"https://api.github.com/users/pixeltrix/received_events","type":"User","site_admin":false},"milestone":null,"comments":6,"created_at":"2015-07-07T18:30:29Z","updated_at":"2015-09-12T16:43:43Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/rails/rails/pulls/20797","html_url":"https://github.com/rails/rails/pull/20797","diff_url":"https://github.com/rails/rails/pull/20797.diff","patch_url":"https://github.com/rails/rails/pull/20797.patch"},"body":"Similar to https://github.com/rails/rails/commit/341698ed40e023898d9f9d1f5c163d4d2cab4832 but generalized at the `url_for` level. Which mean it it will also protect controllers and `link_to`.\r\n\r\nMy main concern with this is that there is a coupling issue by referencing `ActionController::Parameters` in the middle of ActionDispatch.\r\n\r\n@spastorino @rafaelfranca @arthurnn what do you guys think?\r\n\r\ncc @EiNSTeiN- "},"comment":{"url":"https://api.github.com/repos/rails/rails/issues/comments/139796123","html_url":"https://github.com/rails/rails/pull/20797#issuecomment-139796123","issue_url":"https://api.github.com/repos/rails/rails/issues/20797","id":139796123,"user":{"login":"byroot","id":44640,"avatar_url":"https://avatars.githubusercontent.com/u/44640?v=3","gravatar_id":"","url":"https://api.github.com/users/byroot","html_url":"https://github.com/byroot","followers_url":"https://api.github.com/users/byroot/followers","following_url":"https://api.github.com/users/byroot/following{/other_user}","gists_url":"https://api.github.com/users/byroot/gists{/gist_id}","starred_url":"https://api.github.com/users/byroot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/byroot/subscriptions","organizations_url":"https://api.github.com/users/byroot/orgs","repos_url":"https://api.github.com/users/byroot/repos","events_url":"https://api.github.com/users/byroot/events{/privacy}","received_events_url":"https://api.github.com/users/byroot/received_events","type":"User","site_admin":false},"created_at":"2015-09-12T16:43:43Z","updated_at":"2015-09-12T16:43:43Z","body":"@pixeltrix sorry for the delay, but that's it I updated the code (travis bug as always tests are not run again...).\r\n\r\nI can try to take a stab at the documentation. Where do you think that should be?"}}