From 45cd39c386e1f8c6114984892300e80f9341d431 Mon Sep 17 00:00:00 2001
From: Rish <zrishabhgarg@gmail.com>
Date: Wed, 6 Nov 2019 15:21:33 +0700
Subject: [PATCH] Fixed members settings cleanup for mailgun config

---
 .../api/canary/utils/serializers/output/utils/clean.js    | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/core/server/api/canary/utils/serializers/output/utils/clean.js b/core/server/api/canary/utils/serializers/output/utils/clean.js
index e8fd1986da..7b897712bf 100644
--- a/core/server/api/canary/utils/serializers/output/utils/clean.js
+++ b/core/server/api/canary/utils/serializers/output/utils/clean.js
@@ -142,12 +142,12 @@ const action = (attrs) => {
 const settings = (attrs) => {
     // @NOTE: Admin & Content API return a different format, need to mappers
     // We don't send mail provider API key in settings API if set in config
-    let membersSubscriptionSettings = {};
     if (_.isArray(attrs)) {
-        membersSubscriptionSettings = attrs.find(({key}) => {
+        const membersSubscriptionSettingsObj = attrs.find(({key}) => {
             return key === 'members_subscription_settings';
-        }).value;
-        if (config.get('mailgunApiKey')) {
+        }) || {};
+        const membersSubscriptionSettings = membersSubscriptionSettingsObj.value;
+        if (membersSubscriptionSettings && config.get('mailgunApiKey')) {
             let membersSubscriptionSettingsJson = JSON.parse(membersSubscriptionSettings);
             delete membersSubscriptionSettingsJson.mailgunApiKey;
             attrs.forEach((attr) => {