fix: Group MicrosoftAppId and MicrosoftPassword together in the settings page

Composer/packages/client/src/pages/setting/dialog-settings/index.tsx

@@ -56,7 +56,10 @@ export const DialogSettings: React.FC<RouteComponentProps> = () => {
   };
 
   const handleChange = (result: any) => {
-    saveChangeResult(result);
+    // prevent result was undefined, it will cause error
+    if (result && typeof result === 'object') {
+      saveChangeResult(result);
+    }
   };
 
   const hostedControl = () => (

Composer/packages/server/src/models/settings/defaultSettingManager.ts

@@ -1,7 +1,7 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
-import omit from 'lodash/omit';
+import set from 'lodash/set';
 import { SensitiveProperties } from '@bfc/shared';
 import { UserIdentity } from '@bfc/plugin-loader';
 
@@ -66,7 +66,10 @@ export class DefaultSettingManager extends FileSettingManager {
 
   private filterOutSensitiveValue = (obj: any) => {
     if (obj && typeof obj === 'object') {
-      return omit(obj, SensitiveProperties);
+      SensitiveProperties.map((key) => {
+        set(obj, key, '');
+      });
+      return obj;
     }
   };