jenkins-bot has submitted this change and it was merged.

Change subject: Stage risk_score before signature
......................................................................


Stage risk_score before signature

Now that we're actually rounding off the risk_score, make sure
we do that before we calculate the signature, so we're signing
the same thing we're sending.

Bug: T116667
Change-Id: I9bc28a347651651dbf852fe1c4dfcea68c931b60
---
M adyen_gateway/adyen.adapter.php
1 file changed, 1 insertion(+), 1 deletion(-)

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



diff --git a/adyen_gateway/adyen.adapter.php b/adyen_gateway/adyen.adapter.php
index d70d377..8d96274 100644
--- a/adyen_gateway/adyen.adapter.php
+++ b/adyen_gateway/adyen.adapter.php
@@ -52,9 +52,9 @@
                        'amount',
                        'street',
                        'zip',
+                       'risk_score',
                        'billing_signature',
                        'hpp_signature',
-                       'risk_score',
                );
        }
        

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9bc28a347651651dbf852fe1c4dfcea68c931b60
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/DonationInterface
Gerrit-Branch: master
Gerrit-Owner: Ejegg <[email protected]>
Gerrit-Reviewer: Awight <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to