payload: {"action":"closed","issue":{"url":"https://api.github.com/repos/sul-dlss/dor-services-app/issues/545","repository_url":"https://api.github.com/repos/sul-dlss/dor-services-app","labels_url":"https://api.github.com/repos/sul-dlss/dor-services-app/issues/545/labels{/name}","comments_url":"https://api.github.com/repos/sul-dlss/dor-services-app/issues/545/comments","events_url":"https://api.github.com/repos/sul-dlss/dor-services-app/issues/545/events","html_url":"https://github.com/sul-dlss/dor-services-app/issues/545","id":540007081,"node_id":"MDU6SXNzdWU1NDAwMDcwODE=","number":545,"title":"trap for preservation-client gem errors in dor-services-app and handle accordingly","user":{"login":"ndushay","id":96775,"node_id":"MDQ6VXNlcjk2Nzc1","avatar_url":"https://avatars0.githubusercontent.com/u/96775?v=4","gravatar_id":"","url":"https://api.github.com/users/ndushay","html_url":"https://github.com/ndushay","followers_url":"https://api.github.com/users/ndushay/followers","following_url":"https://api.github.com/users/ndushay/following{/other_user}","gists_url":"https://api.github.com/users/ndushay/gists{/gist_id}","starred_url":"https://api.github.com/users/ndushay/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ndushay/subscriptions","organizations_url":"https://api.github.com/users/ndushay/orgs","repos_url":"https://api.github.com/users/ndushay/repos","events_url":"https://api.github.com/users/ndushay/events{/privacy}","received_events_url":"https://api.github.com/users/ndushay/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":{"login":"justinlittman","id":588335,"node_id":"MDQ6VXNlcjU4ODMzNQ==","avatar_url":"https://avatars1.githubusercontent.com/u/588335?v=4","gravatar_id":"","url":"https://api.github.com/users/justinlittman","html_url":"https://github.com/justinlittman","followers_url":"https://api.github.com/users/justinlittman/followers","following_url":"https://api.github.com/users/justinlittman/following{/other_user}","gists_url":"https://api.github.com/users/justinlittman/gists{/gist_id}","starred_url":"https://api.github.com/users/justinlittman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/justinlittman/subscriptions","organizations_url":"https://api.github.com/users/justinlittman/orgs","repos_url":"https://api.github.com/users/justinlittman/repos","events_url":"https://api.github.com/users/justinlittman/events{/privacy}","received_events_url":"https://api.github.com/users/justinlittman/received_events","type":"User","site_admin":false},"assignees":[{"login":"justinlittman","id":588335,"node_id":"MDQ6VXNlcjU4ODMzNQ==","avatar_url":"https://avatars1.githubusercontent.com/u/588335?v=4","gravatar_id":"","url":"https://api.github.com/users/justinlittman","html_url":"https://github.com/justinlittman","followers_url":"https://api.github.com/users/justinlittman/followers","following_url":"https://api.github.com/users/justinlittman/following{/other_user}","gists_url":"https://api.github.com/users/justinlittman/gists{/gist_id}","starred_url":"https://api.github.com/users/justinlittman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/justinlittman/subscriptions","organizations_url":"https://api.github.com/users/justinlittman/orgs","repos_url":"https://api.github.com/users/justinlittman/repos","events_url":"https://api.github.com/users/justinlittman/events{/privacy}","received_events_url":"https://api.github.com/users/justinlittman/received_events","type":"User","site_admin":false}],"milestone":null,"comments":2,"created_at":"2019-12-19T00:52:10Z","updated_at":"2019-12-20T18:37:10Z","closed_at":"2019-12-20T18:37:09Z","author_association":"MEMBER","body":"dor-services-app uses preservation-client in the following ways:\r\n- shelving API service:\r\n   - uses preservation-client to get content diff for shelved files\r\n- versioning service:\r\n   - uses preservation-client to get current_version\r\n- other?\r\n\r\ndor-services-app code should trap for any errors from preservation-client (e.g. can't connect, not found, generic error, response status other than expected (i.e. not a 200) ) and:\r\n- pass along appropriate error code to dor-services-client (which needs to handle it properly)\r\n- notify honeybadger as appropriate\r\n\r\nSee https://github.com/sul-dlss/preservation-client/blob/master/lib/preservation/client.rb#L30-L39"}}
other: {"actor":{"display_login":"mjgiarlo"}}