fix: gaoc bugsnag alert for email.trim

rudderlabs · Aug 29, 2024 · f0c364f · f0c364f
1 parent e21ebd0
commit f0c364f
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/v0/destinations/google_adwords_offline_conversions/utils.js b/src/v0/destinations/google_adwords_offline_conversions/utils.js
@@ -1,5 +1,6 @@
 const sha256 = require('sha256');
 const SqlString = require('sqlstring');
+const isString = require('lodash/isString');
 const { get, set, cloneDeep } = require('lodash');
 const {
   AbortedError,
@@ -271,7 +272,9 @@ const getAddConversionPayload = (message, Config) => {
 
   const userIdentifierInfo = {
     email:
-      hashUserIdentifier && isDefinedAndNotNull(email) ? sha256(email.trim()).toString() : email,
+      hashUserIdentifier && isString(email) && isDefinedAndNotNull(email)
+        ? sha256(email.trim()).toString()
+        : email,
     phone:
       hashUserIdentifier && isDefinedAndNotNull(phone) ? sha256(phone.trim()).toString() : phone,
     address: buildAndGetAddress(message, hashUserIdentifier),
@@ -367,7 +370,9 @@ const getClickConversionPayloadAndEndpoint = (
 
   const userIdentifierInfo = {
     email:
-      hashUserIdentifier && isDefinedAndNotNull(email) ? sha256(email.trim()).toString() : email,
+      hashUserIdentifier && isString(email) && isDefinedAndNotNull(email)
+        ? sha256(email.trim()).toString()
+        : email,
     phone:
       hashUserIdentifier && isDefinedAndNotNull(phone) ? sha256(phone.trim()).toString() : phone,
   };