{"action":"created","issue":{"url":"https://api.github.com/repos/interledger/five-bells-ledger/issues/242","repository_url":"https://api.github.com/repos/interledger/five-bells-ledger","labels_url":"https://api.github.com/repos/interledger/five-bells-ledger/issues/242/labels{/name}","comments_url":"https://api.github.com/repos/interledger/five-bells-ledger/issues/242/comments","events_url":"https://api.github.com/repos/interledger/five-bells-ledger/issues/242/events","html_url":"https://github.com/interledger/five-bells-ledger/pull/242","id":153901768,"number":242,"title":"[TASK] Add switch to disable notification signing (RILP-478)","user":{"login":"MatthewPhinney","id":4961057,"avatar_url":"https://avatars.githubusercontent.com/u/4961057?v=3","gravatar_id":"","url":"https://api.github.com/users/MatthewPhinney","html_url":"https://github.com/MatthewPhinney","followers_url":"https://api.github.com/users/MatthewPhinney/followers","following_url":"https://api.github.com/users/MatthewPhinney/following{/other_user}","gists_url":"https://api.github.com/users/MatthewPhinney/gists{/gist_id}","starred_url":"https://api.github.com/users/MatthewPhinney/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MatthewPhinney/subscriptions","organizations_url":"https://api.github.com/users/MatthewPhinney/orgs","repos_url":"https://api.github.com/users/MatthewPhinney/repos","events_url":"https://api.github.com/users/MatthewPhinney/events{/privacy}","received_events_url":"https://api.github.com/users/MatthewPhinney/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2016-05-10T00:53:08Z","updated_at":"2016-05-10T21:05:15Z","closed_at":"2016-05-10T20:10:10Z","pull_request":{"url":"https://api.github.com/repos/interledger/five-bells-ledger/pulls/242","html_url":"https://github.com/interledger/five-bells-ledger/pull/242","diff_url":"https://github.com/interledger/five-bells-ledger/pull/242.diff","patch_url":"https://github.com/interledger/five-bells-ledger/pull/242.patch"},"body":"cc @gip @nguyenripple"},"comment":{"url":"https://api.github.com/repos/interledger/five-bells-ledger/issues/comments/218291157","html_url":"https://github.com/interledger/five-bells-ledger/pull/242#issuecomment-218291157","issue_url":"https://api.github.com/repos/interledger/five-bells-ledger/issues/242","id":218291157,"user":{"login":"MatthewPhinney","id":4961057,"avatar_url":"https://avatars.githubusercontent.com/u/4961057?v=3","gravatar_id":"","url":"https://api.github.com/users/MatthewPhinney","html_url":"https://github.com/MatthewPhinney","followers_url":"https://api.github.com/users/MatthewPhinney/followers","following_url":"https://api.github.com/users/MatthewPhinney/following{/other_user}","gists_url":"https://api.github.com/users/MatthewPhinney/gists{/gist_id}","starred_url":"https://api.github.com/users/MatthewPhinney/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MatthewPhinney/subscriptions","organizations_url":"https://api.github.com/users/MatthewPhinney/orgs","repos_url":"https://api.github.com/users/MatthewPhinney/repos","events_url":"https://api.github.com/users/MatthewPhinney/events{/privacy}","received_events_url":"https://api.github.com/users/MatthewPhinney/received_events","type":"User","site_admin":false},"created_at":"2016-05-10T21:05:15Z","updated_at":"2016-05-10T21:05:15Z","body":"I just mean, in the current implementation, https notifications are signed and ws notifications are not. Adding signatures to non-wss notifications could be another task. But for this PR, I didn't see any benefit to introducing another flag to change http-notification signature behavior."}}