Mwalker has submitted this change and it was merged.
Change subject: parameterize postback url
......................................................................
parameterize postback url
Deployment:
* Add postback_url and test_postback_url to the config files.
Change-Id: I3891765b308f804bc625517b48bcec4a40e680c9
---
M legacy-paypal/PaypalIPNListener.php
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Mwalker: Verified; Looks good to me, approved
Jgreen: Looks good to me, but someone else must approve
diff --git a/legacy-paypal/PaypalIPNListener.php
b/legacy-paypal/PaypalIPNListener.php
index eb8dc7a..063cbf1 100644
--- a/legacy-paypal/PaypalIPNListener.php
+++ b/legacy-paypal/PaypalIPNListener.php
@@ -343,9 +343,9 @@
*/
public function ipn_verify( $post_data ) {
// url to respond to paypal with verification response
- $postback_url = 'https://www.paypal.com/cgi-bin/webscr'; //
should this be configurable?
+ $postback_url = $this->postback_url;
if ( isset( $post_data[ 'test_ipn' ] )) {
- $postback_url =
'https://www.sandbox.paypal.com/cgi-bin/webscr';
+ $postback_url = $this->test_postback_url;
}
if( array_key_exists( "receiver_email", $post_data ) &&
$post_data["receiver_email"] != $this->receiver_email ){
--
To view, visit https://gerrit.wikimedia.org/r/146961
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I3891765b308f804bc625517b48bcec4a40e680c9
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/fundraising/PaymentsListeners
Gerrit-Branch: master
Gerrit-Owner: Awight <[email protected]>
Gerrit-Reviewer: Awight <[email protected]>
Gerrit-Reviewer: Ejegg <[email protected]>
Gerrit-Reviewer: Jgreen <[email protected]>
Gerrit-Reviewer: Katie Horn <[email protected]>
Gerrit-Reviewer: Mwalker <[email protected]>
Gerrit-Reviewer: Ssmith <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
