Fix issue with notify-cfn's failure case

echo-bravo-yahoo · echo-bravo-yahoo · commit bc1e2e2e141c · 2019-03-07T14:56:24.000-08:00
diff --git a/lambdas/cfn-cognito-user-pools-client-settings/notify-cfn.js b/lambdas/cfn-cognito-user-pools-client-settings/notify-cfn.js
@@ -32,7 +32,7 @@ function notifyCFN ({ event, context, responseStatus, responseData, error, physi
   
   // lastRequestId = event.RequestId
 
-  return new Promise(resolve => {
+  return new Promise((resolve, reject) => {
     let errorMessage = error ? error.message || error.error || error.errorMessage || error : ''
 
     // if there's no response message, default it based on presence of an error
diff --git a/lambdas/cfn-cognito-user-pools-domain/notify-cfn.js b/lambdas/cfn-cognito-user-pools-domain/notify-cfn.js
@@ -32,7 +32,7 @@ function notifyCFN ({ event, context, responseStatus, responseData, error, physi
   
   // lastRequestId = event.RequestId
 
-  return new Promise(resolve => {
+  return new Promise((resolve, reject) => {
     let errorMessage = error ? error.message || error.error || error.errorMessage || error : ''
 
     // if there's no response message, default it based on presence of an error
