{"action":"created","issue":{"url":"https://api.github.com/repos/mozilla/balrog/issues/274","repository_url":"https://api.github.com/repos/mozilla/balrog","labels_url":"https://api.github.com/repos/mozilla/balrog/issues/274/labels{/name}","comments_url":"https://api.github.com/repos/mozilla/balrog/issues/274/comments","events_url":"https://api.github.com/repos/mozilla/balrog/issues/274/events","html_url":"https://github.com/mozilla/balrog/pull/274","id":212706530,"number":274,"title":"Bug: 1287536. Returning the exceptions from __heartbeat with 502 and a message. r=bhearsum.","user":{"login":"tyagi-iiitv","id":8291574,"avatar_url":"https://avatars2.githubusercontent.com/u/8291574?v=3","gravatar_id":"","url":"https://api.github.com/users/tyagi-iiitv","html_url":"https://github.com/tyagi-iiitv","followers_url":"https://api.github.com/users/tyagi-iiitv/followers","following_url":"https://api.github.com/users/tyagi-iiitv/following{/other_user}","gists_url":"https://api.github.com/users/tyagi-iiitv/gists{/gist_id}","starred_url":"https://api.github.com/users/tyagi-iiitv/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tyagi-iiitv/subscriptions","organizations_url":"https://api.github.com/users/tyagi-iiitv/orgs","repos_url":"https://api.github.com/users/tyagi-iiitv/repos","events_url":"https://api.github.com/users/tyagi-iiitv/events{/privacy}","received_events_url":"https://api.github.com/users/tyagi-iiitv/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":9,"created_at":"2017-03-08T11:16:41Z","updated_at":"2017-03-15T14:54:14Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mozilla/balrog/pulls/274","html_url":"https://github.com/mozilla/balrog/pull/274","diff_url":"https://github.com/mozilla/balrog/pull/274.diff","patch_url":"https://github.com/mozilla/balrog/pull/274.patch"},"body":""},"comment":{"url":"https://api.github.com/repos/mozilla/balrog/issues/comments/286766819","html_url":"https://github.com/mozilla/balrog/pull/274#issuecomment-286766819","issue_url":"https://api.github.com/repos/mozilla/balrog/issues/274","id":286766819,"user":{"login":"mostlygeek","id":83972,"avatar_url":"https://avatars1.githubusercontent.com/u/83972?v=3","gravatar_id":"","url":"https://api.github.com/users/mostlygeek","html_url":"https://github.com/mostlygeek","followers_url":"https://api.github.com/users/mostlygeek/followers","following_url":"https://api.github.com/users/mostlygeek/following{/other_user}","gists_url":"https://api.github.com/users/mostlygeek/gists{/gist_id}","starred_url":"https://api.github.com/users/mostlygeek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mostlygeek/subscriptions","organizations_url":"https://api.github.com/users/mostlygeek/orgs","repos_url":"https://api.github.com/users/mostlygeek/repos","events_url":"https://api.github.com/users/mostlygeek/events{/privacy}","received_events_url":"https://api.github.com/users/mostlygeek/received_events","type":"User","site_admin":false},"created_at":"2017-03-15T14:54:14Z","updated_at":"2017-03-15T14:54:14Z","body":"Not sure if I'm too late to the game with some guidance here. \r\n\r\nFor balrog, I think it's enough for __heartbeat__ to just check the database when it's called. If the app can't talk to the database, 502 with an error message, \"can't connect to db\" (or something). A full error/stack trace into sentry would be an extra bonus but it's not necessary.\r\n\r\nIf we add more dependent services in the future just add another check to __heartbeat__. \r\n\r\nAlso, I recommending adding a Cache-Control header of 30 seconds to 1 minute to prevent an easy DDOS against the endpoint. We have nginx in front of balrog with a microcache. \r\n"}}
{"actor":{"display_login":"mostlygeek"}}