use null check

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

@@ -12,17 +12,17 @@ const removeNullPropertyFromObject = require('../../../lib/util/removeNullProper
 
 const validator = {
   generalSetting: [
-    body('restrictGuestMode').if(value => value.exists()).isString().isIn([
+    body('restrictGuestMode').if(value => value != null).isString().isIn([
       'Deny', 'Readonly',
     ]),
-    body('pageCompleteDeletionAuthority').if(value => value.exists()).isString().isIn([
+    body('pageCompleteDeletionAuthority').if(value => value != null).isString().isIn([
       'anyOne', 'adminOnly', 'adminAndAuthor',
     ]),
-    body('hideRestrictedByOwner').if(value => value.exists()).isBoolean(),
-    body('hideRestrictedByGroup').if(value => value.exists()).isBoolean(),
+    body('hideRestrictedByOwner').if(value => value != null).isBoolean(),
+    body('hideRestrictedByGroup').if(value => value != null).isBoolean(),
   ],
   authenticationSetting: [
-    body('isEnabled').if(value => value.exists()).isBoolean(),
+    body('isEnabled').if(value => value != null).isBoolean(),
     body('authId').isString().isIn([
       'local', 'ldap', 'saml', 'oidc', 'basic', 'google', 'github', 'twitter',
     ]),
@@ -31,65 +31,65 @@ const validator = {
     body('registrationMode').isString().isIn([
       'Open', 'Restricted', 'Closed',
     ]),
-    body('registrationWhiteList').if(value => value.exists()).isArray().customSanitizer((value, { req }) => {
+    body('registrationWhiteList').if(value => value != null).isArray().customSanitizer((value, { req }) => {
       return value.filter(email => email !== '');
     }),
   ],
   ldapAuth: [
-    body('serverUrl').if(value => value.exists()).isString(),
-    body('isUserBind').if(value => value.exists()).isBoolean(),
-    body('ldapBindDN').if(value => value.exists()).isString(),
-    body('ldapBindDNPassword').if(value => value.exists()).isString(),
-    body('ldapSearchFilter').if(value => value.exists()).isString(),
-    body('ldapAttrMapUsername').if(value => value.exists()).isString(),
-    body('isSameUsernameTreatedAsIdenticalUser').if(value => value.exists()).isBoolean(),
-    body('ldapAttrMapMail').if(value => value.exists()).isString(),
-    body('ldapAttrMapName').if(value => value.exists()).isString(),
-    body('ldapGroupSearchBase').if(value => value.exists()).isString(),
-    body('ldapGroupSearchFilter').if(value => value.exists()).isString(),
-    body('ldapGroupDnProperty').if(value => value.exists()).isString(),
+    body('serverUrl').if(value => value != null).isString(),
+    body('isUserBind').if(value => value != null).isBoolean(),
+    body('ldapBindDN').if(value => value != null).isString(),
+    body('ldapBindDNPassword').if(value => value != null).isString(),
+    body('ldapSearchFilter').if(value => value != null).isString(),
+    body('ldapAttrMapUsername').if(value => value != null).isString(),
+    body('isSameUsernameTreatedAsIdenticalUser').if(value => value != null).isBoolean(),
+    body('ldapAttrMapMail').if(value => value != null).isString(),
+    body('ldapAttrMapName').if(value => value != null).isString(),
+    body('ldapGroupSearchBase').if(value => value != null).isString(),
+    body('ldapGroupSearchFilter').if(value => value != null).isString(),
+    body('ldapGroupDnProperty').if(value => value != null).isString(),
   ],
   samlAuth: [
-    body('entryPoint').if(value => value.exists()).isString(),
-    body('issuer').if(value => value.exists()).isString(),
-    body('cert').if(value => value.exists()).isString(),
-    body('attrMapId').if(value => value.exists()).isString(),
-    body('attrMapUsername').if(value => value.exists()).isString(),
-    body('attrMapMail').if(value => value.exists()).isString(),
-    body('attrMapFirstName').if(value => value.exists()).isString(),
-    body('attrMapLastName').if(value => value.exists()).isString(),
-    body('isSameUsernameTreatedAsIdenticalUser').if(value => value.exists()).isBoolean(),
-    body('isSameEmailTreatedAsIdenticalUser').if(value => value.exists()).isBoolean(),
-    body('ABLCRule').if(value => value.exists()).isString(),
+    body('entryPoint').if(value => value != null).isString(),
+    body('issuer').if(value => value != null).isString(),
+    body('cert').if(value => value != null).isString(),
+    body('attrMapId').if(value => value != null).isString(),
+    body('attrMapUsername').if(value => value != null).isString(),
+    body('attrMapMail').if(value => value != null).isString(),
+    body('attrMapFirstName').if(value => value != null).isString(),
+    body('attrMapLastName').if(value => value != null).isString(),
+    body('isSameUsernameTreatedAsIdenticalUser').if(value => value != null).isBoolean(),
+    body('isSameEmailTreatedAsIdenticalUser').if(value => value != null).isBoolean(),
+    body('ABLCRule').if(value => value != null).isString(),
   ],
   oidcAuth: [
-    body('oidcProviderName').if(value => value.exists()).isString(),
-    body('oidcIssuerHost').if(value => value.exists()).isString(),
-    body('oidcClientId').if(value => value.exists()).isString(),
-    body('oidcClientSecret').if(value => value.exists()).isString(),
-    body('oidcAttrMapId').if(value => value.exists()).isString(),
-    body('oidcAttrMapUserName').if(value => value.exists()).isString(),
-    body('oidcAttrMapEmail').if(value => value.exists()).isString(),
-    body('isSameUsernameTreatedAsIdenticalUser').if(value => value.exists()).isBoolean(),
-    body('isSameEmailTreatedAsIdenticalUser').if(value => value.exists()).isBoolean(),
+    body('oidcProviderName').if(value => value != null).isString(),
+    body('oidcIssuerHost').if(value => value != null).isString(),
+    body('oidcClientId').if(value => value != null).isString(),
+    body('oidcClientSecret').if(value => value != null).isString(),
+    body('oidcAttrMapId').if(value => value != null).isString(),
+    body('oidcAttrMapUserName').if(value => value != null).isString(),
+    body('oidcAttrMapEmail').if(value => value != null).isString(),
+    body('isSameUsernameTreatedAsIdenticalUser').if(value => value != null).isBoolean(),
+    body('isSameEmailTreatedAsIdenticalUser').if(value => value != null).isBoolean(),
   ],
   basicAuth: [
-    body('isSameUsernameTreatedAsIdenticalUser').if(value => value.exists()).isBoolean(),
+    body('isSameUsernameTreatedAsIdenticalUser').if(value => value != null).isBoolean(),
   ],
   googleOAuth: [
-    body('googleClientId').if(value => value.exists()).isString(),
-    body('googleClientSecret').if(value => value.exists()).isString(),
-    body('isSameUsernameTreatedAsIdenticalUser').if(value => value.exists()).isBoolean(),
+    body('googleClientId').if(value => value != null).isString(),
+    body('googleClientSecret').if(value => value != null).isString(),
+    body('isSameUsernameTreatedAsIdenticalUser').if(value => value != null).isBoolean(),
   ],
   githubOAuth: [
-    body('githubClientId').if(value => value.exists()).isString(),
-    body('githubClientSecret').if(value => value.exists()).isString(),
-    body('isSameUsernameTreatedAsIdenticalUser').if(value => value.exists()).isBoolean(),
+    body('githubClientId').if(value => value != null).isString(),
+    body('githubClientSecret').if(value => value != null).isString(),
+    body('isSameUsernameTreatedAsIdenticalUser').if(value => value != null).isBoolean(),
   ],
   twitterOAuth: [
-    body('twitterConsumerKey').if(value => value.exists()).isString(),
-    body('twitterConsumerSecret').if(value => value.exists()).isString(),
-    body('isSameUsernameTreatedAsIdenticalUser').if(value => value.exists()).isBoolean(),
+    body('twitterConsumerKey').if(value => value != null).isString(),
+    body('twitterConsumerSecret').if(value => value != null).isString(),
+    body('isSameUsernameTreatedAsIdenticalUser').if(value => value != null).isBoolean(),
   ],
 };