{"action":"created","issue":{"url":"https://api.github.com/repos/Expensify/Bedrock/issues/409","repository_url":"https://api.github.com/repos/Expensify/Bedrock","labels_url":"https://api.github.com/repos/Expensify/Bedrock/issues/409/labels{/name}","comments_url":"https://api.github.com/repos/Expensify/Bedrock/issues/409/comments","events_url":"https://api.github.com/repos/Expensify/Bedrock/issues/409/events","html_url":"https://github.com/Expensify/Bedrock/pull/409","id":309213169,"number":409,"title":"[HOLD] Fix edge case for large payloads","user":{"login":"coleaeason","id":8662825,"avatar_url":"https://avatars2.githubusercontent.com/u/8662825?v=4","gravatar_id":"","url":"https://api.github.com/users/coleaeason","html_url":"https://github.com/coleaeason","followers_url":"https://api.github.com/users/coleaeason/followers","following_url":"https://api.github.com/users/coleaeason/following{/other_user}","gists_url":"https://api.github.com/users/coleaeason/gists{/gist_id}","starred_url":"https://api.github.com/users/coleaeason/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/coleaeason/subscriptions","organizations_url":"https://api.github.com/users/coleaeason/orgs","repos_url":"https://api.github.com/users/coleaeason/repos","events_url":"https://api.github.com/users/coleaeason/events{/privacy}","received_events_url":"https://api.github.com/users/coleaeason/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":{"login":"coleaeason","id":8662825,"avatar_url":"https://avatars2.githubusercontent.com/u/8662825?v=4","gravatar_id":"","url":"https://api.github.com/users/coleaeason","html_url":"https://github.com/coleaeason","followers_url":"https://api.github.com/users/coleaeason/followers","following_url":"https://api.github.com/users/coleaeason/following{/other_user}","gists_url":"https://api.github.com/users/coleaeason/gists{/gist_id}","starred_url":"https://api.github.com/users/coleaeason/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/coleaeason/subscriptions","organizations_url":"https://api.github.com/users/coleaeason/orgs","repos_url":"https://api.github.com/users/coleaeason/repos","events_url":"https://api.github.com/users/coleaeason/events{/privacy}","received_events_url":"https://api.github.com/users/coleaeason/received_events","type":"User","site_admin":false},"assignees":[{"login":"coleaeason","id":8662825,"avatar_url":"https://avatars2.githubusercontent.com/u/8662825?v=4","gravatar_id":"","url":"https://api.github.com/users/coleaeason","html_url":"https://github.com/coleaeason","followers_url":"https://api.github.com/users/coleaeason/followers","following_url":"https://api.github.com/users/coleaeason/following{/other_user}","gists_url":"https://api.github.com/users/coleaeason/gists{/gist_id}","starred_url":"https://api.github.com/users/coleaeason/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/coleaeason/subscriptions","organizations_url":"https://api.github.com/users/coleaeason/orgs","repos_url":"https://api.github.com/users/coleaeason/repos","events_url":"https://api.github.com/users/coleaeason/events{/privacy}","received_events_url":"https://api.github.com/users/coleaeason/received_events","type":"User","site_admin":false}],"milestone":null,"comments":0,"created_at":"2018-03-28T03:59:34Z","updated_at":"2018-03-28T18:31:11Z","closed_at":"2018-03-28T18:30:35Z","author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/Expensify/Bedrock/pulls/409","html_url":"https://github.com/Expensify/Bedrock/pull/409","diff_url":"https://github.com/Expensify/Bedrock/pull/409.diff","patch_url":"https://github.com/Expensify/Bedrock/pull/409.patch"},"body":"@tylerkaraszewski Please review. \r\n\r\nAs we discussed, we shouldn't ASSERT on this non-catastrophic case where we don't send our entire payload, then try to remove only the part of it that we did send. I think the comment accurately describes my fix here, but let me know if you had something different in mind or a reason this won't work.\r\n"},"comment":{"url":"https://api.github.com/repos/Expensify/Bedrock/issues/comments/376989684","html_url":"https://github.com/Expensify/Bedrock/pull/409#issuecomment-376989684","issue_url":"https://api.github.com/repos/Expensify/Bedrock/issues/409","id":376989684,"user":{"login":"coleaeason","id":8662825,"avatar_url":"https://avatars2.githubusercontent.com/u/8662825?v=4","gravatar_id":"","url":"https://api.github.com/users/coleaeason","html_url":"https://github.com/coleaeason","followers_url":"https://api.github.com/users/coleaeason/followers","following_url":"https://api.github.com/users/coleaeason/following{/other_user}","gists_url":"https://api.github.com/users/coleaeason/gists{/gist_id}","starred_url":"https://api.github.com/users/coleaeason/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/coleaeason/subscriptions","organizations_url":"https://api.github.com/users/coleaeason/orgs","repos_url":"https://api.github.com/users/coleaeason/repos","events_url":"https://api.github.com/users/coleaeason/events{/privacy}","received_events_url":"https://api.github.com/users/coleaeason/received_events","type":"User","site_admin":false},"created_at":"2018-03-28T18:31:11Z","updated_at":"2018-03-28T18:31:11Z","author_association":"CONTRIBUTOR","body":"We're gonna just close this until it happens again, as it's not at all clear what happened to cause us to hit this ASSERT and reproducing this edge case may be rather difficult and time consuming. "}}
{"actor":{"display_login":"coleaeason"}}