fix: stop accessing response text multiple times (#195)

As text is a stream it can only be accessed once otherwise it would error.
opencrvs · Jul 9, 2024 · 7f3902d · 7f3902d
1 parent dfc63d5
commit 7f3902d
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/src/api/notification/sms-service.ts b/src/api/notification/sms-service.ts
@@ -80,15 +80,13 @@ export async function sendSMS(
   }
 
   const responseBody = await response.text()
-  logger.info(`Response from Infobip: ${JSON.stringify(responseBody)}`)
   if (!response.ok) {
-    logger.error(
-      `Failed to send sms to ${recipient}. Reason: ${response.text()}`
-    )
+    logger.error(`Failed to send sms to ${recipient}. Reason: ${responseBody}`)
     throw internal(
-      `Failed to send notification to ${recipient}. Reason: ${response.text()}`
+      `Failed to send notification to ${recipient}. Reason: ${responseBody}`
     )
   }
+  logger.info(`Response from Infobip: ${JSON.stringify(responseBody)}`)
 }
 
 const compileMessages = async (