From bfe6f4a16302e839393e65634b72b051f9e562da Mon Sep 17 00:00:00 2001 From: Laurent Destailleur Date: Tue, 20 Aug 2019 20:32:17 +0200 Subject: [PATCH] Fix RESTRICT_ON_API --- htdocs/admin/dav.php | 4 +++- htdocs/api/admin/index.php | 6 +++--- htdocs/api/index.php | 8 ++++---- htdocs/dav/fileserver.php | 8 ++++---- htdocs/langs/en_US/admin.lang | 4 ++-- 5 files changed, 16 insertions(+), 14 deletions(-) diff --git a/htdocs/admin/dav.php b/htdocs/admin/dav.php index f4fbcbbdd59..032c4f07e56 100644 --- a/htdocs/admin/dav.php +++ b/htdocs/admin/dav.php @@ -86,7 +86,9 @@ if ($action == 'edit') print ''; $tooltiphelp = (($langs->trans($key.'Tooltip') != $key.'Tooltip') ? $langs->trans($key.'Tooltip') : ''); - print $form->textwithpicto($langs->trans($key), $tooltiphelp); + $label = $langs->trans($key); + if ($key == 'DAV_RESTICT_ON_IP') $label = $langs->trans("RESTRICT_ON_IP"); + print $form->textwithpicto($label, $tooltiphelp); print ''; if ($key == 'DAV_ALLOW_PRIVATE_DIR') { diff --git a/htdocs/api/admin/index.php b/htdocs/api/admin/index.php index 979ce497ab1..cbf18f92222 100644 --- a/htdocs/api/admin/index.php +++ b/htdocs/api/admin/index.php @@ -80,7 +80,7 @@ if ($action == 'setproductionmode') if ($action == 'save') { - dolibarr_set_const($db, 'API_RESTICT_ON_IP', GETPOST('API_RESTICT_ON_IP', 'alpha')); + dolibarr_set_const($db, 'API_RESTRICT_ON_IP', GETPOST('API_RESTRICT_ON_IP', 'alpha')); } @@ -130,8 +130,8 @@ print ' '; print ''; print ''; -print ''.$langs->trans("RESTICT_API_ON_IP").''; -print ''; +print ''.$langs->trans("RESTRICT_API_ON_IP").''; +print ''; print ''; print ''; print ''; diff --git a/htdocs/api/index.php b/htdocs/api/index.php index 1f1235571cc..c63a376e806 100644 --- a/htdocs/api/index.php +++ b/htdocs/api/index.php @@ -207,16 +207,16 @@ if (! empty($reg[1]) && $reg[1] == 'explorer' && ($reg[2] == '/swagger.json' || if (! empty($reg[1]) && ($reg[1] != 'explorer' || ($reg[2] != '/swagger.json' && $reg[2] != '/resources.json' && preg_match('/^\/(swagger|resources)\.json\/(.+)$/', $reg[2], $regbis) && $regbis[2] != 'root'))) { // Restrict API to some IPs - if (! empty($conf->global->API_RESTICT_ON_IP)) + if (! empty($conf->global->API_RESTRICT_ON_IP)) { - $allowedip=explode(' ', $conf->global->API_RESTICT_ON_IP); + $allowedip=explode(' ', $conf->global->API_RESTRICT_ON_IP); $ipremote = getUserRemoteIP(); if (! in_array($ipremote, $allowedip)) { - dol_syslog('Remote ip is '.$ipremote.', not into list '.$conf->global->API_RESTICT_ON_IP); + dol_syslog('Remote ip is '.$ipremote.', not into list '.$conf->global->API_RESTRICT_ON_IP); print 'API not allowed from the IP '.$ipremote; header('HTTP/1.1 503 API not allowed from your IP '.$ipremote); - //print $conf->global->API_RESTICT_ON_IP; + //print $conf->global->API_RESTRICT_ON_IP; exit(0); } } diff --git a/htdocs/dav/fileserver.php b/htdocs/dav/fileserver.php index 4a38511305e..dae6565bc93 100644 --- a/htdocs/dav/fileserver.php +++ b/htdocs/dav/fileserver.php @@ -56,16 +56,16 @@ if (empty($conf->dav->enabled)) // Restrict API to some IPs -if (! empty($conf->global->DAV_RESTICT_ON_IP)) +if (! empty($conf->global->DAV_RESTRICT_ON_IP)) { - $allowedip=explode(' ', $conf->global->DAV_RESTICT_ON_IP); + $allowedip=explode(' ', $conf->global->DAV_RESTRICT_ON_IP); $ipremote = getUserRemoteIP(); if (! in_array($ipremote, $allowedip)) { - dol_syslog('Remote ip is '.$ipremote.', not into list '.$conf->global->DAV_RESTICT_ON_IP); + dol_syslog('Remote ip is '.$ipremote.', not into list '.$conf->global->DAV_RESTRICT_ON_IP); print 'DAV not allowed from the IP '.$ipremote; header('HTTP/1.1 503 DAV not allowed from your IP '.$ipremote); - //print $conf->global->DAV_RESTICT_ON_IP; + //print $conf->global->DAV_RESTRICT_ON_IP; exit(0); } } diff --git a/htdocs/langs/en_US/admin.lang b/htdocs/langs/en_US/admin.lang index 726ba624434..ef5d6388f6e 100644 --- a/htdocs/langs/en_US/admin.lang +++ b/htdocs/langs/en_US/admin.lang @@ -1932,6 +1932,6 @@ DeleteEmailCollector=Delete email collector ConfirmDeleteEmailCollector=Are you sure you want to delete this email collector? RecipientEmailsWillBeReplacedWithThisValue=Recipient emails will be always replaced with this value AtLeastOneDefaultBankAccountMandatory=At least 1 default bank account must be defined -RESTICT_API_ON_IP=Allow available APIs to some host IP only (wildcard not allowed, use space between values). Empty means every hosts can use the available APIs. -RESTICT_ON_IP=Allow access to some host IP only (wildcard not allowed, use space between values). Empty means every hosts can access. +RESTRICT_API_ON_IP=Allow available APIs to some host IP only (wildcard not allowed, use space between values). Empty means every hosts can use the available APIs. +RESTRICT_ON_IP=Allow access to some host IP only (wildcard not allowed, use space between values). Empty means every hosts can access. BaseOnSabeDavVersion=Based on the library SabreDAV version \ No newline at end of file