diff options
author | Horst Knorr <gpgmailencrypt@gmx.de> | 2018-01-30 21:42:37 +0300 |
---|---|---|
committer | Horst Knorr <gpgmailencrypt@gmx.de> | 2018-01-30 21:42:37 +0300 |
commit | 78a02478981891b98a2cbaa15e4bad83dd1c1324 (patch) | |
tree | c8b83f4185a861d27cd2000209f7e9897be39476 | |
parent | cf98f74ba7aeae52219dd01047121664c41aa884 (diff) |
code for supervisor mode removed, because functionality exists in official nextcloud app "impersonate" with better GUI
-rw-r--r-- | README.md | 7 | ||||
-rw-r--r-- | ajax/settings.php | 8 | ||||
-rw-r--r-- | appinfo/update.php | 2 | ||||
-rw-r--r-- | js/settings.js | 7 | ||||
-rw-r--r-- | lib/helper.php | 2 | ||||
-rw-r--r-- | lib/user_sql.php | 16 | ||||
-rw-r--r-- | templates/admin.php | 13 |
7 files changed, 1 insertions, 54 deletions
@@ -57,11 +57,6 @@ Password changing is disabled by default, but can be enabled in the Admin area. Caution: user_sql does not recreate password salts, which imposes a security risk. Password salts should be newly generated whenever the password changes. -Supervision can be enabled under supervisor settings. Supervision allows one -specified user to login into any account. Use supervisor username and target -username separated by ';' to login as target user using supervisor's password -(ex. superuser;user). - The column autocomplete works only for MySQL and PostgreSQL database which is used to validate form data. If you use other database use *occ* command to set the application config parameters with domain suffix. @@ -84,8 +79,6 @@ For example to set 'sql_hostname' parameter in default domain use: - col_email - col_gethome - set_active_invert -- set_supervisor -- supervisor - set_allow_pwchange - set_default_domain - set_strip_domain diff --git a/ajax/settings.php b/ajax/settings.php index cdbf2eb..77de46f 100644 --- a/ajax/settings.php +++ b/ajax/settings.php @@ -134,10 +134,6 @@ if(isset($_POST['appname']) && ($_POST['appname'] === 'user_sql') && isset($_POS { \OC::$server->getConfig()->setAppValue('user_sql', 'set_active_invert_'.$domain, 'true'); } - elseif($param === 'set_supervisor') - { - \OC::$server->getConfig()->setAppValue('user_sql', 'set_supervisor_'.$domain, 'true'); - } elseif($param === 'set_enable_gethome') { \OC::$server->getConfig()->setAppValue('user_sql', 'set_enable_gethome_'.$domain, 'true'); @@ -160,10 +156,6 @@ if(isset($_POST['appname']) && ($_POST['appname'] === 'user_sql') && isset($_POS { \OC::$server->getConfig()->setAppValue('user_sql', 'set_active_invert_'.$domain, 'false'); } - elseif($param === 'set_supervisor') - { - \OC::$server->getConfig()->setAppValue('user_sql', 'set_supervisor_'.$domain, 'false'); - } elseif($param === 'set_enable_gethome') { \OC::$server->getConfig()->setAppValue('user_sql', 'set_enable_gethome_'.$domain, 'false'); diff --git a/appinfo/update.php b/appinfo/update.php index c5ae8a2..6312a16 100644 --- a/appinfo/update.php +++ b/appinfo/update.php @@ -32,8 +32,6 @@ $params = array('sql_host' => 'sql_hostname', 'sql_column_password' => 'col_password', 'sql_type' => 'sql_driver', 'sql_column_active' => 'col_active', - 'sql_column_supervisor' => 'supervisor', - 'sql_supervisor' => 'set_supervisor', 'strip_domain' => 'set_strip_domain', 'default_domain' => 'set_default_domain', 'crypt_type' => 'set_crypt_type', diff --git a/js/settings.js b/js/settings.js index 7455299..f9f6b44 100644 --- a/js/settings.js +++ b/js/settings.js @@ -322,13 +322,6 @@ user_sql.loadDomainSettings = function(domain) else $('#' + key).prop('checked', false); } - else if(key == 'set_supervisor') - { - if(data.settings[key] == 'true') - $('#' + key).prop('checked', true); - else - $('#' + key).prop('checked', false); - } else if(key == 'set_enable_gethome') { if(data.settings[key] == 'true') diff --git a/lib/helper.php b/lib/helper.php index 6375e26..0c0f782 100644 --- a/lib/helper.php +++ b/lib/helper.php @@ -59,8 +59,6 @@ class Helper { 'col_email', 'col_gethome', 'set_active_invert', - 'set_supervisor', - 'supervisor', 'set_allow_pwchange', 'set_default_domain', 'set_strip_domain', diff --git a/lib/user_sql.php b/lib/user_sql.php index 41a819e..eba5d38 100644 --- a/lib/user_sql.php +++ b/lib/user_sql.php @@ -389,21 +389,6 @@ class OC_USER_SQL extends BackendUtility implements \OCP\IUserBackend, $uid = $this -> doUserDomainMapping($uid); - $superuid = $this -> settings['supervisor']; - if($uid !== $superuid && $this -> settings['set_supervisor'] === 'true' && substr($uid, 0, strlen($superuid)) === $superuid) - { - $row = $this -> helper -> runQuery('getPass', array('uid' => $superuid)); - if($row === false) - { - Util::writeLog('OC_USER_SQL', "Got no row, return false", Util::DEBUG); - return false; - } - Util::writeLog('OC_USER_SQL', "Logging in as supervisor", Util::DEBUG); - $db_pass = $row[$this -> settings['col_password']]; - $uid = explode(';', $uid)[1]; - } - else - { $row = $this -> helper -> runQuery('getPass', array('uid' => $uid)); if($row === false) { @@ -411,7 +396,6 @@ class OC_USER_SQL extends BackendUtility implements \OCP\IUserBackend, return false; } $db_pass = $row[$this -> settings['col_password']]; - } Util::writeLog('OC_USER_SQL', "Encrypting and checking password", Util::DEBUG); diff --git a/templates/admin.php b/templates/admin.php index 8cd6e05..d7f7048 100644 --- a/templates/admin.php +++ b/templates/admin.php @@ -22,8 +22,7 @@ $cfgClass = 'section'; <li><a id="sqlEmailSettings" href="#sql-3"><?php p($l -> t('E-Mail Settings')); ?></a></li> <li><a id="sqlDomainSettings" href="#sql-4"><?php p($l -> t('Domain Settings')); ?></a></li> <li><a id="sqlGethomeSettings" href="#sql-5"><?php p($l -> t('getHome Settings')); ?></a></li> - <li><a id="sqlSupervisorSettings" href="#sql-6"><?php p($l -> t('Supervisor Settings')); ?></a></li> - <li><a id="sqlGroupsSettings" href="#sql-7"><?php p($l -> t('Groups Settings')); ?></a></li> + <li><a id="sqlGroupsSettings" href="#sql-6"><?php p($l -> t('Groups Settings')); ?></a></li> </ul> <fieldset id="sql-1"> @@ -160,17 +159,7 @@ $cfgClass = 'section'; <em><?php p($l -> t('You can use the placeholders %%u to specify the user ID (before appending the default domain), %%ud to specify the user ID (after appending the default domain) and %%d to specify the default domain')); ?></em></p> </fieldset> - <fieldset id="sql-6"> - <p><label for="set_supervisor"><?php p($l -> t('Enable supervisor')); ?></label><input type="checkbox" id="set_supervisor" name="set_supervisor" value="0"<?php - if($_['set_supervisor']) - p(' checked'); - ?> /><br> - <p><label for="supervisor"><?php p($l -> t('Supervisor username')); ?></label><input type="text" id="supervisor" name="supervisor" value="<?php p($_['supervisor']); ?>" /></p> - <em><?php p($l -> t("Use supervisor username and target username separated by ';' to login as target user using supervisor's password (ex. superuser;user).")); ?></em></p> - </fieldset> - - <fieldset id="sql-7"> <p><label for="sql_group_table"><?php p($l -> t('Table')); ?></label><input type="text" id="sql_group_table" name="sql_group_table" value="<?php p($_['sql_group_table']); ?>" /></p> <p><label for="col_group_username"><?php p($l -> t('Username Column')); ?></label><input type="text" id="col_group_username" name="col_group_username" value="<?php p($_['col_group_username']); ?>" /></p> |