jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/356970 )

Change subject: Alert on Amazon message verification failure
......................................................................


Alert on Amazon message verification failure

We should know about these ASAP

Bug: T166087
Change-Id: Icd269abe5706d82fcf7aa5c81f4baeeae5ea5996
---
M PaymentProviders/Amazon/AmazonListener.php
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Mepps: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/PaymentProviders/Amazon/AmazonListener.php 
b/PaymentProviders/Amazon/AmazonListener.php
index 6d7a831..96ead27 100644
--- a/PaymentProviders/Amazon/AmazonListener.php
+++ b/PaymentProviders/Amazon/AmazonListener.php
@@ -6,6 +6,7 @@
 use SmashPig\Core\Listeners\ListenerDataException;
 use SmashPig\Core\Listeners\RestListener;
 use SmashPig\Core\Logging\Logger;
+use SmashPig\Maintenance\TestFailMail;
 
 /**
  * Uses the Amazon SDK to parse incoming IPN messages
@@ -57,6 +58,7 @@
                        // because of how RestListener is implemented and 
because we only
                        // process one message per request
                        // Bad form, but it would be odd to hold this till 
doMessageSecurity
+                       Logger::alert( 'Error validating Amazon message. 
Firewall problem?' );
                        throw new ListenerSecurityException( $ex->getMessage() 
);
                }
                $messageValues = $amazonHandlerMessage->toArray();

-- 
To view, visit https://gerrit.wikimedia.org/r/356970
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Icd269abe5706d82fcf7aa5c81f4baeeae5ea5996
Gerrit-PatchSet: 2
Gerrit-Project: wikimedia/fundraising/SmashPig
Gerrit-Branch: master
Gerrit-Owner: Ejegg <[email protected]>
Gerrit-Reviewer: AndyRussG <[email protected]>
Gerrit-Reviewer: Awight <[email protected]>
Gerrit-Reviewer: Cdentinger <[email protected]>
Gerrit-Reviewer: Eileen <[email protected]>
Gerrit-Reviewer: Katie Horn <[email protected]>
Gerrit-Reviewer: Mepps <[email protected]>
Gerrit-Reviewer: XenoRyet <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to