{"action":"created","issue":{"url":"https://api.github.com/repos/drastik/com.drastikbydesign.stripe/issues/140","repository_url":"https://api.github.com/repos/drastik/com.drastikbydesign.stripe","labels_url":"https://api.github.com/repos/drastik/com.drastikbydesign.stripe/issues/140/labels{/name}","comments_url":"https://api.github.com/repos/drastik/com.drastikbydesign.stripe/issues/140/comments","events_url":"https://api.github.com/repos/drastik/com.drastikbydesign.stripe/issues/140/events","html_url":"https://github.com/drastik/com.drastikbydesign.stripe/pull/140","id":162543570,"number":140,"title":"Multiple Stripe processor accounts cause IPN fail","user":{"login":"agh1","id":1682375,"avatar_url":"https://avatars.githubusercontent.com/u/1682375?v=3","gravatar_id":"","url":"https://api.github.com/users/agh1","html_url":"https://github.com/agh1","followers_url":"https://api.github.com/users/agh1/followers","following_url":"https://api.github.com/users/agh1/following{/other_user}","gists_url":"https://api.github.com/users/agh1/gists{/gist_id}","starred_url":"https://api.github.com/users/agh1/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/agh1/subscriptions","organizations_url":"https://api.github.com/users/agh1/orgs","repos_url":"https://api.github.com/users/agh1/repos","events_url":"https://api.github.com/users/agh1/events{/privacy}","received_events_url":"https://api.github.com/users/agh1/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-06-27T20:42:13Z","updated_at":"2016-06-28T19:18:26Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/drastik/com.drastikbydesign.stripe/pulls/140","html_url":"https://github.com/drastik/com.drastikbydesign.stripe/pull/140","diff_url":"https://github.com/drastik/com.drastikbydesign.stripe/pull/140.diff","patch_url":"https://github.com/drastik/com.drastikbydesign.stripe/pull/140.patch"},"body":"When there are multiple Stripe payment processors on a site, the webhook just gets the API key from the first one.  That means that the \"event\" lookup fails for the other Stripe instances.\r\n\r\nI added an optional `ppid` URL parameter that lets you specify the payment processor ID.\r\n\r\nI know this is not the new preferred URL format per @eileenmcnaughton's [comment on CRM-18107](https://issues.civicrm.org/jira/browse/CRM-18107?focusedCommentId=86710&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-86710), but I wanted to introduce a modest change that leaves the URL alone for the 99% of instances where there's only one Stripe account and/or no recurring donations.\r\n\r\nSeparately, I got annoyed dealing with all the other page markup when testing the webhook calls, so I made sure that the page run exits rather than returning.  `CRM_Core_Error::fatal()` exits anyway, so I got rid of the superfluous `exit()` calls following it."},"comment":{"url":"https://api.github.com/repos/drastik/com.drastikbydesign.stripe/issues/comments/229153495","html_url":"https://github.com/drastik/com.drastikbydesign.stripe/pull/140#issuecomment-229153495","issue_url":"https://api.github.com/repos/drastik/com.drastikbydesign.stripe/issues/140","id":229153495,"user":{"login":"agh1","id":1682375,"avatar_url":"https://avatars.githubusercontent.com/u/1682375?v=3","gravatar_id":"","url":"https://api.github.com/users/agh1","html_url":"https://github.com/agh1","followers_url":"https://api.github.com/users/agh1/followers","following_url":"https://api.github.com/users/agh1/following{/other_user}","gists_url":"https://api.github.com/users/agh1/gists{/gist_id}","starred_url":"https://api.github.com/users/agh1/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/agh1/subscriptions","organizations_url":"https://api.github.com/users/agh1/orgs","repos_url":"https://api.github.com/users/agh1/repos","events_url":"https://api.github.com/users/agh1/events{/privacy}","received_events_url":"https://api.github.com/users/agh1/received_events","type":"User","site_admin":false},"created_at":"2016-06-28T19:18:26Z","updated_at":"2016-06-28T19:18:26Z","body":"@mattwire Interesting--I hadn't thought about that issue.  I think these two are parallel solutions to separate problems that come up with multiple accounts.  #127 handles contacts who have made contributions through two or more Stripe accounts, while #140 handles webhook notifications that don't indicate which account they're coming from.\r\n\r\nI think both should be incorporated--we just hadn't noticed the issue with multiple customer accounts because it's a new site where few people have made contributions through both Stripe accounts."}}
{"actor":{"display_login":"agh1"}}