From 8529485837ebb2b46e3ae7cefd89e4043652a825 Mon Sep 17 00:00:00 2001 From: Girish Ramakrishnan Date: Fri, 4 Dec 2020 18:45:52 -0800 Subject: [PATCH] sftp: require admin by default (breaking change) --- migrations/20201204081504-settings-sftp-requireAdmin.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/migrations/20201204081504-settings-sftp-requireAdmin.js b/migrations/20201204081504-settings-sftp-requireAdmin.js index 282c7d6ed..c707945f9 100644 --- a/migrations/20201204081504-settings-sftp-requireAdmin.js +++ b/migrations/20201204081504-settings-sftp-requireAdmin.js @@ -4,11 +4,11 @@ exports.up = function(db, callback) { db.all('SELECT * FROM settings WHERE name=?', ['platform_config'], function (error, results) { let value; if (error || results.length === 0) { - value = { sftp: { requireAdmin: true } }; // new installations + value = { sftp: { requireAdmin: true } }; } else { value = JSON.parse(results[0].value); if (!value.sftp) value.sftp = {}; - value.sftp.requireAdmin = false; // existing installation, backward compat + value.sftp.requireAdmin = true; } db.runSql('UPDATE settings SET value=? WHERE name=?', [ JSON.stringify(value), 'platform_config' ], callback);