{"action":"created","issue":{"url":"https://api.github.com/repos/Yoast/wordpress-seo/issues/5446","repository_url":"https://api.github.com/repos/Yoast/wordpress-seo","labels_url":"https://api.github.com/repos/Yoast/wordpress-seo/issues/5446/labels{/name}","comments_url":"https://api.github.com/repos/Yoast/wordpress-seo/issues/5446/comments","events_url":"https://api.github.com/repos/Yoast/wordpress-seo/issues/5446/events","html_url":"https://github.com/Yoast/wordpress-seo/pull/5446","id":173485449,"number":5446,"title":"Started to add HTML escaping for WordPress.XSS.EscapeOutput sniff audit","user":{"login":"Rarst","id":737584,"avatar_url":"https://avatars.githubusercontent.com/u/737584?v=3","gravatar_id":"","url":"https://api.github.com/users/Rarst","html_url":"https://github.com/Rarst","followers_url":"https://api.github.com/users/Rarst/followers","following_url":"https://api.github.com/users/Rarst/following{/other_user}","gists_url":"https://api.github.com/users/Rarst/gists{/gist_id}","starred_url":"https://api.github.com/users/Rarst/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Rarst/subscriptions","organizations_url":"https://api.github.com/users/Rarst/orgs","repos_url":"https://api.github.com/users/Rarst/repos","events_url":"https://api.github.com/users/Rarst/events{/privacy}","received_events_url":"https://api.github.com/users/Rarst/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":{"login":"Rarst","id":737584,"avatar_url":"https://avatars.githubusercontent.com/u/737584?v=3","gravatar_id":"","url":"https://api.github.com/users/Rarst","html_url":"https://github.com/Rarst","followers_url":"https://api.github.com/users/Rarst/followers","following_url":"https://api.github.com/users/Rarst/following{/other_user}","gists_url":"https://api.github.com/users/Rarst/gists{/gist_id}","starred_url":"https://api.github.com/users/Rarst/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Rarst/subscriptions","organizations_url":"https://api.github.com/users/Rarst/orgs","repos_url":"https://api.github.com/users/Rarst/repos","events_url":"https://api.github.com/users/Rarst/events{/privacy}","received_events_url":"https://api.github.com/users/Rarst/received_events","type":"User","site_admin":false},"assignees":[{"login":"Rarst","id":737584,"avatar_url":"https://avatars.githubusercontent.com/u/737584?v=3","gravatar_id":"","url":"https://api.github.com/users/Rarst","html_url":"https://github.com/Rarst","followers_url":"https://api.github.com/users/Rarst/followers","following_url":"https://api.github.com/users/Rarst/following{/other_user}","gists_url":"https://api.github.com/users/Rarst/gists{/gist_id}","starred_url":"https://api.github.com/users/Rarst/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Rarst/subscriptions","organizations_url":"https://api.github.com/users/Rarst/orgs","repos_url":"https://api.github.com/users/Rarst/repos","events_url":"https://api.github.com/users/Rarst/events{/privacy}","received_events_url":"https://api.github.com/users/Rarst/received_events","type":"User","site_admin":false}],"milestone":null,"comments":5,"created_at":"2016-08-26T15:53:28Z","updated_at":"2016-11-10T08:57:26Z","closed_at":"2016-11-07T12:30:35Z","pull_request":{"url":"https://api.github.com/repos/Yoast/wordpress-seo/pulls/5446","html_url":"https://github.com/Yoast/wordpress-seo/pull/5446","diff_url":"https://github.com/Yoast/wordpress-seo/pull/5446.diff","patch_url":"https://github.com/Yoast/wordpress-seo/pull/5446.patch"},"body":"## Summary\n\nThis PR can be summarized in the following changelog entry:\n\nImproved output escaping for security and protection from XSS.\n## Relevant technical choices:\n- added appropriate escaping to satisfy WordPress.XSS.EscapeOutput sniff, where possible without significant changes to code\n## Test instructions\n\nThis PR can be tested by following these steps:\n- these should mostly be code–safe, complex cases would be too hard to handle during large pass\n\nSee #2763\n"},"comment":{"url":"https://api.github.com/repos/Yoast/wordpress-seo/issues/comments/259635446","html_url":"https://github.com/Yoast/wordpress-seo/pull/5446#issuecomment-259635446","issue_url":"https://api.github.com/repos/Yoast/wordpress-seo/issues/5446","id":259635446,"user":{"login":"atimmer","id":584693,"avatar_url":"https://avatars.githubusercontent.com/u/584693?v=3","gravatar_id":"","url":"https://api.github.com/users/atimmer","html_url":"https://github.com/atimmer","followers_url":"https://api.github.com/users/atimmer/followers","following_url":"https://api.github.com/users/atimmer/following{/other_user}","gists_url":"https://api.github.com/users/atimmer/gists{/gist_id}","starred_url":"https://api.github.com/users/atimmer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/atimmer/subscriptions","organizations_url":"https://api.github.com/users/atimmer/orgs","repos_url":"https://api.github.com/users/atimmer/repos","events_url":"https://api.github.com/users/atimmer/events{/privacy}","received_events_url":"https://api.github.com/users/atimmer/received_events","type":"User","site_admin":false},"created_at":"2016-11-10T08:57:26Z","updated_at":"2016-11-10T08:57:26Z","body":"I wholeheartedly agree with adding the proper escaping and the automatic testing. We are just not comfortable merging and releasing such a big change. We would merge a PR per file. Boyscouting this is our own way of tackling this problem."}}
{"actor":{"display_login":"atimmer"}}