GitHub PullRequestReviewCommentEvent at 2014-05-02 15:24:17
Repository: rails
Actor: rafaelfranca
Payload: {"comment":{"url":"https://api.github.com/repos/rails/rails/pulls/comments/12233405","id":12233405,"diff_hunk":"@@ -461,17 +461,27 @@ def test_should_allow_all_methods_without_token\n \n class CustomAuthenticityParamControllerTest < ActionController::TestCase\n   def setup\n-    super\n-    ActionController::Base.request_forgery_protection_token = :custom_token_name\n+    @logger = ActiveSupport::LogSubscriber::TestHelper::MockLogger.new\n+    @token = \"foobar\"\n+    ActionController::Base.request_forgery_protection_token = @token","path":"actionpack/test/controller/request_forgery_protection_test.rb","position":8,"original_position":8,"commit_id":"55b5a0df2bc69437a21cb4e56c863f06e93cafd6","original_commit_id":"55b5a0df2bc69437a21cb4e56c863f06e93cafd6","user":{"login":"rafaelfranca","id":47848,"avatar_url":"https://avatars.githubusercontent.com/u/47848?","gravatar_id":"0525b332aafb83307b32d9747a93de03","url":"https://api.github.com/users/rafaelfranca","html_url":"https://github.com/rafaelfranca","followers_url":"https://api.github.com/users/rafaelfranca/followers","following_url":"https://api.github.com/users/rafaelfranca/following{/other_user}","gists_url":"https://api.github.com/users/rafaelfranca/gists{/gist_id}","starred_url":"https://api.github.com/users/rafaelfranca/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rafaelfranca/subscriptions","organizations_url":"https://api.github.com/users/rafaelfranca/orgs","repos_url":"https://api.github.com/users/rafaelfranca/repos","events_url":"https://api.github.com/users/rafaelfranca/events{/privacy}","received_events_url":"https://api.github.com/users/rafaelfranca/received_events","type":"User","site_admin":false},"body":"We need to change back this value in the teardown method. Also seems we are not using `@token` anywhere so we can remove it.","created_at":"2014-05-02T15:24:17Z","updated_at":"2014-05-02T15:24:17Z","html_url":"https://github.com/rails/rails/pull/14945#discussion_r12233405","pull_request_url":"https://api.github.com/repos/rails/rails/pulls/14945","_links":{"self":{"href":"https://api.github.com/repos/rails/rails/pulls/comments/12233405"},"html":{"href":"https://github.com/rails/rails/pull/14945#discussion_r12233405"},"pull_request":{"href":"https://api.github.com/repos/rails/rails/pulls/14945"}}}}