{"action":"created","issue":{"url":"https://api.github.com/repos/alphagov/notifications-api/issues/1492","repository_url":"https://api.github.com/repos/alphagov/notifications-api","labels_url":"https://api.github.com/repos/alphagov/notifications-api/issues/1492/labels{/name}","comments_url":"https://api.github.com/repos/alphagov/notifications-api/issues/1492/comments","events_url":"https://api.github.com/repos/alphagov/notifications-api/issues/1492/events","html_url":"https://github.com/alphagov/notifications-api/pull/1492","id":282126697,"number":1492,"title":"[DO NOT MERGE] Add MMG_INBOUND_SMS_AUTH config variable and auth check","user":{"login":"klssmith","id":12881990,"avatar_url":"https://avatars3.githubusercontent.com/u/12881990?v=4","gravatar_id":"","url":"https://api.github.com/users/klssmith","html_url":"https://github.com/klssmith","followers_url":"https://api.github.com/users/klssmith/followers","following_url":"https://api.github.com/users/klssmith/following{/other_user}","gists_url":"https://api.github.com/users/klssmith/gists{/gist_id}","starred_url":"https://api.github.com/users/klssmith/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/klssmith/subscriptions","organizations_url":"https://api.github.com/users/klssmith/orgs","repos_url":"https://api.github.com/users/klssmith/repos","events_url":"https://api.github.com/users/klssmith/events{/privacy}","received_events_url":"https://api.github.com/users/klssmith/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-12-14T14:52:30Z","updated_at":"2017-12-14T14:53:22Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/alphagov/notifications-api/pulls/1492","html_url":"https://github.com/alphagov/notifications-api/pull/1492","diff_url":"https://github.com/alphagov/notifications-api/pull/1492.diff","patch_url":"https://github.com/alphagov/notifications-api/pull/1492.patch"},"body":"Checks authentication header value on inbound SMS requests from\r\nMMG  against a list of allowed API keys set in the application\r\nconfig.\r\n\r\nAt the moment, we're only logging the attempts without aborting the\r\nrequests. Once this is rolled out to production and we've checked\r\nthe logs we'll switch on the aborts and add the tests for 401 and 403\r\nresponses.\r\n\r\nThis work has already been done for Firetext in a previous PR:\r\nhttps://github.com/alphagov/notifications-api/pull/1409"},"comment":{"url":"https://api.github.com/repos/alphagov/notifications-api/issues/comments/351733065","html_url":"https://github.com/alphagov/notifications-api/pull/1492#issuecomment-351733065","issue_url":"https://api.github.com/repos/alphagov/notifications-api/issues/1492","id":351733065,"user":{"login":"klssmith","id":12881990,"avatar_url":"https://avatars3.githubusercontent.com/u/12881990?v=4","gravatar_id":"","url":"https://api.github.com/users/klssmith","html_url":"https://github.com/klssmith","followers_url":"https://api.github.com/users/klssmith/followers","following_url":"https://api.github.com/users/klssmith/following{/other_user}","gists_url":"https://api.github.com/users/klssmith/gists{/gist_id}","starred_url":"https://api.github.com/users/klssmith/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/klssmith/subscriptions","organizations_url":"https://api.github.com/users/klssmith/orgs","repos_url":"https://api.github.com/users/klssmith/repos","events_url":"https://api.github.com/users/klssmith/events{/privacy}","received_events_url":"https://api.github.com/users/klssmith/received_events","type":"User","site_admin":false},"created_at":"2017-12-14T14:53:22Z","updated_at":"2017-12-14T14:53:22Z","author_association":"CONTRIBUTOR","body":"This will need the right credentials to be added before it can be merged."}}
{"actor":{"display_login":"klssmith"}}