{"action":"created","issue":{"url":"https://api.github.com/repos/Workiva/w_service/issues/63","labels_url":"https://api.github.com/repos/Workiva/w_service/issues/63/labels{/name}","comments_url":"https://api.github.com/repos/Workiva/w_service/issues/63/comments","events_url":"https://api.github.com/repos/Workiva/w_service/issues/63/events","html_url":"https://github.com/Workiva/w_service/pull/63","id":104131033,"number":63,"title":"CP-938 CSRF interceptor updates","user":{"login":"jayudey-wf","id":4429043,"avatar_url":"https://avatars.githubusercontent.com/u/4429043?v=3","gravatar_id":"","url":"https://api.github.com/users/jayudey-wf","html_url":"https://github.com/jayudey-wf","followers_url":"https://api.github.com/users/jayudey-wf/followers","following_url":"https://api.github.com/users/jayudey-wf/following{/other_user}","gists_url":"https://api.github.com/users/jayudey-wf/gists{/gist_id}","starred_url":"https://api.github.com/users/jayudey-wf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jayudey-wf/subscriptions","organizations_url":"https://api.github.com/users/jayudey-wf/orgs","repos_url":"https://api.github.com/users/jayudey-wf/repos","events_url":"https://api.github.com/users/jayudey-wf/events{/privacy}","received_events_url":"https://api.github.com/users/jayudey-wf/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-08-31T20:44:56Z","updated_at":"2015-09-01T18:44:46Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Workiva/w_service/pulls/63","html_url":"https://github.com/Workiva/w_service/pull/63","diff_url":"https://github.com/Workiva/w_service/pull/63.diff","patch_url":"https://github.com/Workiva/w_service/pull/63.patch"},"body":"## Issue\r\n- #58 \r\n- #57 \r\n\r\n## Changes\r\n**Source:**\r\n- added null check to throw error if the csrf value is null\r\n- remove csrf header if the method doesn't require it\r\n\r\n**Tests:**\r\n- added unit test coverage\r\n\r\n## Areas of Regression\r\n- csrf interceptor\r\n\r\n## Testing\r\n- passing CI\r\n- example still works as expected\r\n- in example `GET`, `HEAD`, `OPTIONS` and `TRACE` requests shouldn't have a csrf header\r\n\r\n## Code Review\r\n@trentgrover-wf\r\n@maxwellpeterson-wf\r\n@evanweible-wf\r\n@dustinlessard-wf"},"comment":{"url":"https://api.github.com/repos/Workiva/w_service/issues/comments/136824584","html_url":"https://github.com/Workiva/w_service/pull/63#issuecomment-136824584","issue_url":"https://api.github.com/repos/Workiva/w_service/issues/63","id":136824584,"user":{"login":"evanweible-wf","id":1738457,"avatar_url":"https://avatars.githubusercontent.com/u/1738457?v=3","gravatar_id":"","url":"https://api.github.com/users/evanweible-wf","html_url":"https://github.com/evanweible-wf","followers_url":"https://api.github.com/users/evanweible-wf/followers","following_url":"https://api.github.com/users/evanweible-wf/following{/other_user}","gists_url":"https://api.github.com/users/evanweible-wf/gists{/gist_id}","starred_url":"https://api.github.com/users/evanweible-wf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/evanweible-wf/subscriptions","organizations_url":"https://api.github.com/users/evanweible-wf/orgs","repos_url":"https://api.github.com/users/evanweible-wf/repos","events_url":"https://api.github.com/users/evanweible-wf/events{/privacy}","received_events_url":"https://api.github.com/users/evanweible-wf/received_events","type":"User","site_admin":false},"created_at":"2015-09-01T18:44:46Z","updated_at":"2015-09-01T18:44:46Z","body":"+10\r\n- Tests pass\r\n- Example works as expected\r\n  - DELETE, PATCH, POST, PUT requests all had the CSRF header added\r\n  - GET, HEAD, and OPTIONS requests did not have the CSRF header added\r\n  - _TRACE request cannot be tested in the example because browsers can't send TRACE requests_"}}