Merge pull request #169 from alphamanuscript/128-fix-flw-transfer-not…

…ification-error Fix error processing flutterwave transfer notification
alphamanuscript · Oct 28, 2020 · 912066e · 912066e
2 parents 1102219 + a0762df
commit 912066e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/server/src/core/payment/flutterwave-payment-provider.ts b/server/src/core/payment/flutterwave-payment-provider.ts
@@ -63,10 +63,13 @@ interface FlutterwaveTransactionResponse {
 function extractTransactionInfo(data: FlutterwaveTransactionInfo): ProviderTransactionInfo {
   const status: TransactionStatus = data.status === 'successful' ? 'success' :
       data.status === 'failed' ? 'failed' : 'pending';
+
+  // phone number comes in in 07... format, strip the 0
+  const flwPhone = data.customer?.phone_number?.substring(1);
 
   return {
     userData: {
-      phone: `254${data.customer.phone_number.substring(1)}` // use internal instead of local format
+      phone: flwPhone ? `254${flwPhone}` : '' // convert to internal 254 format
     },
     status,
     amount: data.amount,

diff --git a/server/src/webhooks/flutterwave.ts b/server/src/webhooks/flutterwave.ts
@@ -11,6 +11,7 @@ flutterwaveRoutes.post('/', (req: AppRequest, res) => {
     .then(_ => res.status(200).send())
     .catch(e => {
       console.error('Flutterwave notification error', e);
+      console.error('Notification payload', JSON.stringify(req.body, null, 2));
       res.status(400).send();
     });
 });