Merge pull request #1716 from weseek/fix/registrationWhiteList-save-empty-array

registrationWhiteList-save-empty-array

src/client/js/services/AdminLocalSecurityContainer.js

@@ -65,8 +65,7 @@ export default class AdminLocalSecurityContainer extends Container {
    * update local security setting
    */
   async updateLocalSecuritySetting() {
-    let { registrationWhiteList } = this.state;
-    registrationWhiteList = Array.isArray(registrationWhiteList) ? registrationWhiteList : registrationWhiteList.split('\n');
+    const { registrationWhiteList } = this.state;
     const response = await this.appContainer.apiv3.put('/security-setting/local-setting', {
       registrationMode: this.state.registrationMode,
       registrationWhiteList,

src/server/routes/apiv3/security-setting.js

@@ -31,7 +31,9 @@ const validator = {
     body('registrationMode').isString().isIn([
       'Open', 'Restricted', 'Closed',
     ]),
-    body('registrationWhiteList').if((value, { req }) => req.body.registrationWhiteList).isArray(),
+    body('registrationWhiteList').if((value, { req }) => req.body.registrationWhiteList).isArray().customSanitizer((value, { req }) => {
+      return value.filter(email => email !== '');
+    }),
   ],
   ldapAuth: [
     body('serverUrl').if((value, { req }) => req.body.serverUrl).isString(),