sftp: requireAdmin is true by default

for existing installs, it is off for backward compatibility
This commit is contained in:
Girish Ramakrishnan
2020-12-04 00:22:21 -08:00
parent 59c51c5747
commit 3b38bb5d33
3 changed files with 22 additions and 3 deletions

View File

@@ -579,8 +579,7 @@ function loadSftpConfig(req, res, next) {
addons.getServicesConfig('sftp', function (error, service, servicesConfig) {
if (error) return next(new ldap.OperationsError(error.toString()));
const serviceConfig = servicesConfig['sftp'];
req.requireAdmin = serviceConfig && 'requireAdmin' in serviceConfig ? serviceConfig.requireAdmin : true;
req.requireAdmin = servicesConfig['sftp'].requireAdmin;
next();
});