maketank/Froxlor
2
0
Fork 0
Code Pull Requests Releases Activity

implemented SubDomains.update; minor fixes and enhancements in Domains-Command and validateUrl-function

Browse Source 
Signed-off-by: Michael Kaufmann (d00p) <d00p@froxlor.org>
This commit is contained in:
Michael Kaufmann (d00p)
2018-03-05 12:13:01 +01:00
parent ae4a7ff943
commit b2e2590324
6 changed files with 402 additions and 408 deletions
Download Patch File Download Diff File Expand all files Collapse all files

234
customer_domains.php
View File

@@ -299,231 +299,23 @@ if ($page == 'overview') {
} }
} elseif ($action == 'edit' && $id != 0) { } elseif ($action == 'edit' && $id != 0) {
$stmt = Database::prepare("SELECT `d`.*, `pd`.`subcanemaildomain` try {
FROM `" . TABLE_PANEL_DOMAINS . "` `d`, `" . TABLE_PANEL_DOMAINS . "` `pd` $json_result = SubDomains::getLocal($userinfo, array(
WHERE `d`.`customerid` = :customerid 'id' => $id
AND `d`.`id` = :id ))->get();
AND ((`d`.`parentdomainid`!='0' } catch (Exception $e) {
AND `pd`.`id` = `d`.`parentdomainid`) dynamic_error($e->getMessage());
OR (`d`.`parentdomainid`='0' }
AND `pd`.`id` = `d`.`id`)) $result = json_decode($json_result, true)['data'];
AND `d`.`caneditdomain`='1'");
$result = Database::pexecute_first($stmt, array("customerid" => $userinfo['customerid'], "id" => $id));
$alias_stmt = Database::prepare("SELECT COUNT(`id`) AS count FROM `" . TABLE_PANEL_DOMAINS . "` WHERE `aliasdomain`= :aliasdomain");
$alias_check = Database::pexecute_first($alias_stmt, array("aliasdomain" => $result['id']));
$alias_check = $alias_check['count'];
$_doredirect = false;
if (isset($result['customerid']) && $result['customerid'] == $userinfo['customerid']) { if (isset($result['customerid']) && $result['customerid'] == $userinfo['customerid']) {
if (isset($_POST['send']) && $_POST['send'] == 'send') { if (isset($_POST['send']) && $_POST['send'] == 'send') {
if (isset($_POST['url']) && $_POST['url'] != '' && validateUrl($_POST['url'])) { try {
$path = $_POST['url']; SubDomains::getLocal($userinfo, $_POST)->update();
$_doredirect = true; } catch (Exception $e) {
} else { dynamic_error($e->getMessage());
$path = validate($_POST['path'], 'path');
}
if (!preg_match('/^https?\:\/\//', $path) || !validateUrl($path)) {
if (strstr($path, ":") !== FALSE) {
standard_error('pathmaynotcontaincolon');
}
// If path is empty or '/' and 'Use domain name as default value for DocumentRoot path' is enabled in settings,
// set default path to subdomain or domain name
if ((($path == '') || ($path == '/')) && Settings::Get('system.documentroot_use_default_value') == 1) {
$path = makeCorrectDir($userinfo['documentroot'] . '/' . $result['domain']);
} else {
$path = makeCorrectDir($userinfo['documentroot'] . '/' . $path);
}
} else {
$_doredirect = true;
}
$aliasdomain = isset($_POST['alias']) ? intval($_POST['alias']) : 0;
if (isset($_POST['selectserveralias'])) {
$iswildcarddomain = ($_POST['selectserveralias'] == '0') ? '1' : '0';
$wwwserveralias = ($_POST['selectserveralias'] == '1') ? '1' : '0';
} else {
$iswildcarddomain = $result['iswildcarddomain'];
$wwwserveralias = $result['wwwserveralias'];
}
if ($result['parentdomainid'] != '0' && ($result['subcanemaildomain'] == '1' || $result['subcanemaildomain'] == '2') && isset($_POST['isemaildomain'])) {
$isemaildomain = intval($_POST['isemaildomain']);
} else {
$isemaildomain = $result['isemaildomain'];
}
$aliasdomain_check = array('id' => 0);
if ($aliasdomain != 0) {
$aliasdomain_stmt = Database::prepare("SELECT `id` FROM `" . TABLE_PANEL_DOMAINS . "` `d`,`" . TABLE_PANEL_CUSTOMERS . "` `c`
WHERE `d`.`customerid`= :customerid
AND `d`.`aliasdomain` IS NULL
AND `d`.`id`<>`c`.`standardsubdomain`
AND `c`.`customerid`= :customerid
AND `d`.`id`= :id"
);
$aliasdomain_check = Database::pexecute_first($aliasdomain_stmt, array("customerid" => $result['customerid'], "id" => $aliasdomain));
}
if ($aliasdomain_check['id'] != $aliasdomain) {
standard_error('domainisaliasorothercustomer');
}
if (isset($_POST['openbasedir_path']) && $_POST['openbasedir_path'] == '1') {
$openbasedir_path = '1';
} else {
$openbasedir_path = '0';
}
// check whether the customer has chosen its own php-config
if (isset($_POST['phpsettingid'])) {
$phpsettingid = intval($_POST['phpsettingid']);
} else {
$phpsettingid = $result['phpsettingid'];
}
if (isset($_POST['ssl_redirect']) && $_POST['ssl_redirect'] == '1') {
// a ssl-redirect only works if there actually is a
// ssl ip/port assigned to the domain
if (domainHasSslIpPort($id) == true) {
$ssl_redirect = '1';
$_doredirect = true;
} else {
standard_error('sslredirectonlypossiblewithsslipport');
}
} else {
$ssl_redirect = '0';
}
if (isset($_POST['letsencrypt']) && $_POST['letsencrypt'] == '1') {
// let's encrypt only works if there actually is a
// ssl ip/port assigned to the domain
if (domainHasSslIpPort($id) == true) {
$letsencrypt = '1';
} else {
standard_error('letsencryptonlypossiblewithsslipport');
}
} else {
$letsencrypt = '0';
}
// We can't enable let's encrypt for wildcard - domains when using acme-v1
if ($iswildcarddomain == '1' && $letsencrypt == '1' && Settings::Get('system.leapiversion') == '1') {
standard_error('nowildcardwithletsencrypt');
}
// if using acme-v2 we cannot issue wildcard-certificates
// because they currently only support the dns-01 challenge
if ($iswildcarddomain == '0' && $letsencrypt == '1' && Settings::Get('system.leapiversion') == '2') {
standard_error('nowildcardwithletsencryptv2');
}
// Temporarily deactivate ssl_redirect until Let's Encrypt certificate was generated
if ($ssl_redirect > 0 && $letsencrypt == 1 && $result['letsencrypt'] != $letsencrypt) {
$ssl_redirect = 2;
}
// HSTS
$hsts_maxage = isset($_POST['hsts_maxage']) ? (int)$_POST['hsts_maxage'] : 0;
$hsts_sub = isset($_POST['hsts_sub']) && (int)$_POST['hsts_sub'] == 1 ? 1 : 0;
$hsts_preload = isset($_POST['hsts_preload']) && (int)$_POST['hsts_preload'] == 1 ? 1 : 0;
if ($path == '') {
standard_error('patherror');
} else {
if (($result['isemaildomain'] == '1') && ($isemaildomain == '0')) {
$params = array("customerid" => $userinfo['customerid'], "domainid" => $id);
$stmt = Database::prepare("DELETE FROM `" . TABLE_MAIL_USERS . "` WHERE `customerid`= :customerid AND `domainid`= :domainid");
Database::pexecute($stmt, $params);
$stmt = Database::prepare("DELETE FROM `" . TABLE_MAIL_VIRTUAL . "` WHERE `customerid`= :customerid AND `domainid`= :domainid");
Database::pexecute($stmt, $params);
$log->logAction(USR_ACTION, LOG_NOTICE, "automatically deleted mail-table entries for '" . $idna_convert->decode($result['domain']) . "'");
}
if ($_doredirect) {
$redirect = isset($_POST['redirectcode']) ? (int)$_POST['redirectcode'] : false;
updateRedirectOfDomain($id, $redirect);
}
if ($path != $result['documentroot']
|| $isemaildomain != $result['isemaildomain']
|| $wwwserveralias != $result['wwwserveralias']
|| $iswildcarddomain != $result['iswildcarddomain']
|| $aliasdomain != $result['aliasdomain']
|| $openbasedir_path != $result['openbasedir_path']
|| $ssl_redirect != $result['ssl_redirect']
|| $letsencrypt != $result['letsencrypt']
|| $hsts_maxage != $result['hsts']
|| $hsts_sub != $result['hsts_sub']
|| $hsts_preload != $result['hsts_preload']
|| $phpsettingid != $result['phpsettingid']
) {
$log->logAction(USR_ACTION, LOG_INFO, "edited domain '" . $idna_convert->decode($result['domain']) . "'");
$stmt = Database::prepare("UPDATE `" . TABLE_PANEL_DOMAINS . "` SET
`documentroot`= :documentroot,
`isemaildomain`= :isemaildomain,
`wwwserveralias`= :wwwserveralias,
`iswildcarddomain`= :iswildcarddomain,
`aliasdomain`= :aliasdomain,
`openbasedir_path`= :openbasedir_path,
`ssl_redirect`= :ssl_redirect,
`letsencrypt`= :letsencrypt,
`hsts` = :hsts,
`hsts_sub` = :hsts_sub,
`hsts_preload` = :hsts_preload,
`phpsettingid` = :phpsettingid
WHERE `customerid`= :customerid
AND `id`= :id"
);
$params = array(
"documentroot" => $path,
"isemaildomain" => $isemaildomain,
"wwwserveralias" => $wwwserveralias,
"iswildcarddomain" => $iswildcarddomain,
"aliasdomain" => ($aliasdomain != 0 && $alias_check == 0) ? $aliasdomain : null,
"openbasedir_path" => $openbasedir_path,
"ssl_redirect" => $ssl_redirect,
"letsencrypt" => $letsencrypt,
"hsts" => $hsts_maxage,
"hsts_sub" => $hsts_sub,
"hsts_preload" => $hsts_preload,
"phpsettingid" => $phpsettingid,
"customerid" => $userinfo['customerid'],
"id" => $id
);
Database::pexecute($stmt, $params);
if ($result['aliasdomain'] != $aliasdomain) {
// trigger when domain id for alias destination has changed: both for old and new destination
triggerLetsEncryptCSRForAliasDestinationDomain($result['aliasdomain'], $log);
triggerLetsEncryptCSRForAliasDestinationDomain($aliasdomain, $log);
} elseif ($result['wwwserveralias'] != $wwwserveralias || $result['letsencrypt'] != $letsencrypt) {
// or when wwwserveralias or letsencrypt was changed
triggerLetsEncryptCSRForAliasDestinationDomain($aliasdomain, $log);
}
// check whether LE has been disabled, so we remove the certificate
if ($letsencrypt == '0' && $result['letsencrypt'] == '1') {
$del_stmt = Database::prepare("
DELETE FROM `" . TABLE_PANEL_DOMAIN_SSL_SETTINGS . "` WHERE `domainid` = :id
");
Database::pexecute($del_stmt, array(
'id' => $id
));
}
inserttask('1');
// Using nameserver, insert a task which rebuilds the server config
inserttask('4');
}
redirectTo($filename, array('page' => $page, 's' => $s));
} }
redirectTo($filename, array('page' => $page, 's' => $s));
} else { } else {
$result['domain'] = $idna_convert->decode($result['domain']); $result['domain'] = $idna_convert->decode($result['domain']);

121
lib/classes/api/commands/class.Domains.php
View File

@@ -469,66 +469,6 @@ class Domains extends ApiCommand implements ResourceEntity
} elseif ($aliasdomain_check['id'] != $aliasdomain) { } elseif ($aliasdomain_check['id'] != $aliasdomain) {
standard_error('domainisaliasorothercustomer', '', true); standard_error('domainisaliasorothercustomer', '', true);
} else { } else {
/**
*
* @todo how to handle security questions now?
*
* $params = array(
* 'page' => $page,
* 'action' => $action,
* 'domain' => $domain,
* 'customerid' => $customerid,
* 'adminid' => $adminid,
* 'documentroot' => $documentroot,
* 'alias' => $aliasdomain,
* 'isbinddomain' => $isbinddomain,
* 'isemaildomain' => $isemaildomain,
* 'email_only' => $email_only,
* 'subcanemaildomain' => $subcanemaildomain,
* 'caneditdomain' => $caneditdomain,
* 'zonefile' => $zonefile,
* 'dkim' => $dkim,
* 'speciallogfile' => $speciallogfile,
* 'selectserveralias' => $serveraliasoption,
* 'ipandport' => serialize($ipandports),
* 'ssl_redirect' => $ssl_redirect,
* 'ssl_ipandport' => serialize($ssl_ipandports),
* 'phpenabled' => $phpenabled,
* 'openbasedir' => $openbasedir,
* 'phpsettingid' => $phpsettingid,
* 'mod_fcgid_starter' => $mod_fcgid_starter,
* 'mod_fcgid_maxrequests' => $mod_fcgid_maxrequests,
* 'specialsettings' => $specialsettings,
* 'notryfiles' => $notryfiles,
* 'registration_date' => $registration_date,
* 'termination_date' => $termination_date,
* 'issubof' => $issubof,
* 'letsencrypt' => $letsencrypt,
* 'http2' => $http2,
* 'hsts_maxage' => $hsts_maxage,
* 'hsts_sub' => $hsts_sub,
* 'hsts_preload' => $hsts_preload,
* 'ocsp_stapling' => $ocsp_stapling
* );
*
* $security_questions = array(
* 'reallydisablesecuritysetting' => ($openbasedir == '0' && $userinfo['change_serversettings'] == '1'),
* 'reallydocrootoutofcustomerroot' => (substr($documentroot, 0, strlen($customer['documentroot'])) != $customer['documentroot'] && ! preg_match('/^https?\:\/\//', $documentroot))
* );
* $question_nr = 1;
* foreach ($security_questions as $question_name => $question_launch) {
* if ($question_launch !== false) {
* $params[$question_name] = $question_name;
*
* if (! isset($_POST[$question_name]) || $_POST[$question_name] != $question_name) {
* ask_yesno('admin_domain_' . $question_name, $filename, $params, $question_nr);
* }
* }
* $question_nr ++;
* }
*/
$wwwserveralias = ($serveraliasoption == '1') ? '1' : '0'; $wwwserveralias = ($serveraliasoption == '1') ? '1' : '0';
$iswildcarddomain = ($serveraliasoption == '0') ? '1' : '0'; $iswildcarddomain = ($serveraliasoption == '0') ? '1' : '0';
@@ -1087,66 +1027,7 @@ class Domains extends ApiCommand implements ResourceEntity
if ($serveraliasoption != '1' && $serveraliasoption != '2') { if ($serveraliasoption != '1' && $serveraliasoption != '2') {
$serveraliasoption = '0'; $serveraliasoption = '0';
} }
/**
*
* @todo how to handle security questions now?
*
* $params = array(
* 'id' => $id,
* 'page' => $page,
* 'action' => $action,
* 'customerid' => $customerid,
* 'adminid' => $adminid,
* 'documentroot' => $documentroot,
* 'alias' => $aliasdomain,
* 'isbinddomain' => $isbinddomain,
* 'isemaildomain' => $isemaildomain,
* 'email_only' => $email_only,
* 'subcanemaildomain' => $subcanemaildomain,
* 'caneditdomain' => $caneditdomain,
* 'zonefile' => $zonefile,
* 'dkim' => $dkim,
* 'selectserveralias' => $serveraliasoption,
* 'ssl_redirect' => $ssl_redirect,
* 'phpenabled' => $phpenabled,
* 'openbasedir' => $openbasedir,
* 'phpsettingid' => $phpsettingid,
* 'phpsettingsforsubdomains' => $phpfs,
* 'mod_fcgid_starter' => $mod_fcgid_starter,
* 'mod_fcgid_maxrequests' => $mod_fcgid_maxrequests,
* 'specialsettings' => $specialsettings,
* 'specialsettingsforsubdomains' => $ssfs,
* 'notryfiles' => $notryfiles,
* 'registration_date' => $registration_date,
* 'termination_date' => $termination_date,
* 'issubof' => $issubof,
* 'speciallogfile' => $speciallogfile,
* 'speciallogverified' => $speciallogverified,
* 'ipandport' => serialize($ipandports),
* 'ssl_ipandport' => serialize($ssl_ipandports),
* 'letsencrypt' => $letsencrypt,
* 'http2' => $http2,
* 'hsts_maxage' => $hsts_maxage,
* 'hsts_sub' => $hsts_sub,
* 'hsts_preload' => $hsts_preload,
* 'ocsp_stapling' => $ocsp_stapling
* );
*
* $security_questions = array(
* 'reallydisablesecuritysetting' => ($openbasedir == '0' && $userinfo['change_serversettings'] == '1'),
* 'reallydocrootoutofcustomerroot' => (substr($documentroot, 0, strlen($customer['documentroot'])) != $customer['documentroot'] && ! preg_match('/^https?\:\/\//', $documentroot))
* );
* foreach ($security_questions as $question_name => $question_launch) {
* if ($question_launch !== false) {
* $params[$question_name] = $question_name;
* if (! isset($_POST[$question_name]) || $_POST[$question_name] != $question_name) {
* ask_yesno('admin_domain_' . $question_name, $filename, $params);
* }
* }
* }
*/
$wwwserveralias = ($serveraliasoption == '1') ? '1' : '0'; $wwwserveralias = ($serveraliasoption == '1') ? '1' : '0';
$iswildcarddomain = ($serveraliasoption == '0') ? '1' : '0'; $iswildcarddomain = ($serveraliasoption == '0') ? '1' : '0';

334
lib/classes/api/commands/class.SubDomains.php
View File

@@ -62,6 +62,7 @@ class SubDomains extends ApiCommand implements ResourceEntity
$openbasedir_path = $this->getParam('openbasedir_path', true, 1); $openbasedir_path = $this->getParam('openbasedir_path', true, 1);
$phpsettingid = $this->getParam('phpsettingid', true, 0); $phpsettingid = $this->getParam('phpsettingid', true, 0);
$redirectcode = $this->getParam('redirectcode', true, Settings::Get('customredirect.default')); $redirectcode = $this->getParam('redirectcode', true, Settings::Get('customredirect.default'));
$isemaildomain = $this->getParam('isemaildomain', true, 0);
if (Settings::Get('system.use_ssl')) { if (Settings::Get('system.use_ssl')) {
$ssl_redirect = $this->getParam('ssl_redirect', true, 0); $ssl_redirect = $this->getParam('ssl_redirect', true, 0);
$letsencrypt = $this->getParam('letsencrypt', true, 0); $letsencrypt = $this->getParam('letsencrypt', true, 0);
@@ -76,7 +77,7 @@ class SubDomains extends ApiCommand implements ResourceEntity
$hsts_preload = 0; $hsts_preload = 0;
} }
// get needed customer info to reduce the mysql-usage-counter by one // get needed customer info to reduce the subdomain-usage-counter by one
if ($this->isAdmin()) { if ($this->isAdmin()) {
// get customer id // get customer id
$customer_id = $this->getParam('customer_id'); $customer_id = $this->getParam('customer_id');
@@ -115,7 +116,7 @@ class SubDomains extends ApiCommand implements ResourceEntity
if ($completedomain == Settings::Get('system.hostname')) { if ($completedomain == Settings::Get('system.hostname')) {
standard_error('admin_domain_emailsystemhostname', '', true); standard_error('admin_domain_emailsystemhostname', '', true);
} }
// check whether the domain already exists // check whether the domain already exists
$completedomain_stmt = Database::prepare(" $completedomain_stmt = Database::prepare("
SELECT * FROM `" . TABLE_PANEL_DOMAINS . "` SELECT * FROM `" . TABLE_PANEL_DOMAINS . "`
@@ -128,12 +129,12 @@ class SubDomains extends ApiCommand implements ResourceEntity
"domain" => $completedomain, "domain" => $completedomain,
"customerid" => $customer_id "customerid" => $customer_id
), true, true); ), true, true);
if ($completedomain_check) { if ($completedomain_check) {
// no exception so far - domain exists // no exception so far - domain exists
standard_error('domainexistalready', $completedomain, true); standard_error('domainexistalready', $completedomain, true);
} }
// alias domain checked? // alias domain checked?
if ($aliasdomain != 0) { if ($aliasdomain != 0) {
// also check ip/port combination to be the same, #176 // also check ip/port combination to be the same, #176
@@ -160,36 +161,13 @@ class SubDomains extends ApiCommand implements ResourceEntity
triggerLetsEncryptCSRForAliasDestinationDomain($aliasdomain, $this->logger()); triggerLetsEncryptCSRForAliasDestinationDomain($aliasdomain, $this->logger());
} }
// check whether an URL was specified // validate / correct path/url of domain
$_doredirect = false; $path = $this->validateDomainDocumentRoot($path, $url, $customer, $completedomain, $_doredirect);
if (! empty($url) && validateUrl($url)) {
$path = $url;
$_doredirect = true;
} else {
$path = validate($path, 'path', '', '', array(), true);
}
// check whether path is a real path
if (! preg_match('/^https?\:\/\//', $path) || ! validateUrl($path)) {
if (strstr($path, ":") !== false) {
standard_error('pathmaynotcontaincolon', '', true);
}
// If path is empty or '/' and 'Use domain name as default value for DocumentRoot path' is enabled in settings,
// set default path to subdomain or domain name
if ((($path == '') || ($path == '/')) && Settings::Get('system.documentroot_use_default_value') == 1) {
$path = makeCorrectDir($customer['documentroot'] . '/' . $completedomain);
} else {
$path = makeCorrectDir($customer['documentroot'] . '/' . $path);
}
} else {
// no it's not, create a redirect
$_doredirect = true;
}
if ($openbasedir_path != 1) { if ($openbasedir_path != 1) {
$openbasedir_path = 0; $openbasedir_path = 0;
} }
// get main domain for various checks // get main domain for various checks
$domain_stmt = Database::prepare(" $domain_stmt = Database::prepare("
SELECT * FROM `" . TABLE_PANEL_DOMAINS . "` SELECT * FROM `" . TABLE_PANEL_DOMAINS . "`
@@ -214,7 +192,14 @@ class SubDomains extends ApiCommand implements ResourceEntity
// the domain does already exist as main-domain // the domain does already exist as main-domain
standard_error('domainexistalready', $completedomain, true); standard_error('domainexistalready', $completedomain, true);
} }
// if allowed, check for 'is email domain'-flag
if ($domain_check['subcanemaildomain'] == '1' || $domain_check['subcanemaildomain'] == '2') {
$isemaildomain = intval($isemaildomain);
} else {
$isemaildomain = $domain_check['subcanemaildomain'] == '3' ? 1 : 0;
}
if ($ssl_redirect != 0) { if ($ssl_redirect != 0) {
// a ssl-redirect only works if there actually is a // a ssl-redirect only works if there actually is a
// ssl ip/port assigned to the domain // ssl ip/port assigned to the domain
@@ -289,7 +274,7 @@ class SubDomains extends ApiCommand implements ResourceEntity
"parentdomainid" => $domain_check['id'], "parentdomainid" => $domain_check['id'],
"wwwserveralias" => $domain_check['wwwserveralias'] == '1' ? '1' : '0', "wwwserveralias" => $domain_check['wwwserveralias'] == '1' ? '1' : '0',
"iswildcarddomain" => $domain_check['iswildcarddomain'] == '1' ? '1' : '0', "iswildcarddomain" => $domain_check['iswildcarddomain'] == '1' ? '1' : '0',
"isemaildomain" => $domain_check['subcanemaildomain'] == '3' ? '1' : '0', "isemaildomain" => $isemaildomain,
"openbasedir" => $domain_check['openbasedir'], "openbasedir" => $domain_check['openbasedir'],
"openbasedir_path" => $openbasedir_path, "openbasedir_path" => $openbasedir_path,
"phpenabled" => $domain_check['phpenabled'], "phpenabled" => $domain_check['phpenabled'],
@@ -328,7 +313,7 @@ class SubDomains extends ApiCommand implements ResourceEntity
Admins::increaseUsage(($this->isAdmin() ? $customer['adminid'] : $this->getUserDetail('adminid')), 'subdomains_used'); Admins::increaseUsage(($this->isAdmin() ? $customer['adminid'] : $this->getUserDetail('adminid')), 'subdomains_used');
$this->logger()->logAction($this->isAdmin() ? ADM_ACTION : USR_ACTION, LOG_INFO, "[API] added subdomain '" . $completedomain . "'"); $this->logger()->logAction($this->isAdmin() ? ADM_ACTION : USR_ACTION, LOG_INFO, "[API] added subdomain '" . $completedomain . "'");
$result = $this->apiCall('SubDomains.get', array( $result = $this->apiCall('SubDomains.get', array(
'id' => $subdomain_id 'id' => $subdomain_id
)); ));
@@ -373,8 +358,10 @@ class SubDomains extends ApiCommand implements ResourceEntity
} }
if (count($customer_ids) > 0) { if (count($customer_ids) > 0) {
$result_stmt = Database::prepare(" $result_stmt = Database::prepare("
SELECT * FROM `" . TABLE_PANEL_DOMAINS . "` SELECT d.*, pd.`subcanemaildomain`
WHERE " . ($id > 0 ? "`id` = :iddn" : "`domain` = :iddn") . " AND `customerid` IN (:customerids) FROM `" . TABLE_PANEL_DOMAINS . "` d, `" . TABLE_PANEL_DOMAINS . "` pd
WHERE " . ($id > 0 ? "d.`id` = :iddn" : "d.`domain` = :iddn") . " AND d.`customerid` IN (:customerids)
AND ((d.`parentdomainid`!='0' AND pd.`id` = d.`parentdomainid`) OR (d.`parentdomainid`='0' AND pd.`id` = d.`id`))
"); ");
$params = array( $params = array(
'iddn' => ($id <= 0 ? $domainname : $id), 'iddn' => ($id <= 0 ? $domainname : $id),
@@ -385,8 +372,11 @@ class SubDomains extends ApiCommand implements ResourceEntity
} }
} else { } else {
$result_stmt = Database::prepare(" $result_stmt = Database::prepare("
SELECT * FROM `" . TABLE_PANEL_DOMAINS . "` SELECT d.*, pd.`subcanemaildomain`
WHERE " . ($id > 0 ? "`id` = :iddn" : "`domain` = :iddn")); FROM `" . TABLE_PANEL_DOMAINS . "` d, `" . TABLE_PANEL_DOMAINS . "` pd
WHERE " . ($id > 0 ? "d.`id` = :iddn" : "d.`domain` = :iddn") . "
AND ((d.`parentdomainid`!='0' AND pd.`id` = d.`parentdomainid`) OR (d.`parentdomainid`='0' AND pd.`id` = d.`id`))
");
$params = array( $params = array(
'iddn' => ($id <= 0 ? $domainname : $id) 'iddn' => ($id <= 0 ? $domainname : $id)
); );
@@ -396,8 +386,11 @@ class SubDomains extends ApiCommand implements ResourceEntity
throw new Exception("You cannot access this resource", 405); throw new Exception("You cannot access this resource", 405);
} }
$result_stmt = Database::prepare(" $result_stmt = Database::prepare("
SELECT `id`, `customerid`, `domain`, `documentroot`, `isemaildomain`, `parentdomainid`, `aliasdomain`, `caneditdomain` FROM `" . TABLE_PANEL_DOMAINS . "` SELECT d.*, pd.`subcanemaildomain`
WHERE `customerid`= :customerid AND " . ($id > 0 ? "`id` = :iddn" : "`domain` = :iddn")); FROM `" . TABLE_PANEL_DOMAINS . "` d, `" . TABLE_PANEL_DOMAINS . "` pd
WHERE d.`customerid`= :customerid AND " . ($id > 0 ? "d.`id` = :iddn" : "d.`domain` = :iddn") . "
AND ((d.`parentdomainid`!='0' AND pd.`id` = d.`parentdomainid`) OR (d.`parentdomainid`='0' AND pd.`id` = d.`id`))
");
$params = array( $params = array(
'customerid' => $this->getUserDetail('customerid'), 'customerid' => $this->getUserDetail('customerid'),
'iddn' => ($id <= 0 ? $domainname : $id) 'iddn' => ($id <= 0 ? $domainname : $id)
@@ -414,7 +407,225 @@ class SubDomains extends ApiCommand implements ResourceEntity
public function update() public function update()
{ {
throw new Exception("Not available yet.", 501); $id = $this->getParam('id', true, 0);
$dn_optional = ($id <= 0 ? false : true);
$domainname = $this->getParam('domainname', $dn_optional, '');
if ($this->isAdmin() == false && Settings::IsInList('panel.customer_hide_options', 'domains')) {
throw new Exception("You cannot access this resource", 405);
}
$result = $this->apiCall('SubDomains.get', array(
'id' => $id,
'domainname' => $domainname
));
$id = $result['id'];
// parameters
$aliasdomain = $this->getParam('alias', true, 0);
$path = $this->getParam('path', true, $result['documentroot']);
$url = $this->getParam('url', true, '');
// default: 0 = wildcard, 1 = www-alias, 2 = none
$_serveraliasdefault = $result['iswildcarddomain'] == '1' ? 0 : ($result['wwwserveralias'] == '1' ? 1 : 2);
$selectserveralias = $this->getParam('selectserveralias', true, $_serveraliasdefault);
$isemaildomain = $this->getParam('isemaildomain', true, $result['isemaildomain']);
$openbasedir_path = $this->getParam('openbasedir_path', true, $result['openbasedir_path']);
$phpsettingid = $this->getParam('phpsettingid', true, $result['phpsettingid']);
$redirectcode = $this->getParam('redirectcode', true, getDomainRedirectId($id));
if (Settings::Get('system.use_ssl')) {
$ssl_redirect = $this->getParam('ssl_redirect', true, $result['ssl_redirect']);
$letsencrypt = $this->getParam('letsencrypt', true, $result['letsencrypt']);
$hsts_maxage = $this->getParam('hsts_maxage', true, $result['hsts']);
$hsts_sub = $this->getParam('hsts_sub', true, $result['hsts_sub']);
$hsts_preload = $this->getParam('hsts_preload', true, $result['hsts_preload']);
} else {
$ssl_redirect = 0;
$letsencrypt = 0;
$hsts_maxage = 0;
$hsts_sub = 0;
$hsts_preload = 0;
}
// get needed customer info to reduce the subdomain-usage-counter by one
if ($this->isAdmin()) {
// get customer id
$customer_id = $this->getParam('customer_id');
$customer = $this->apiCall('Customers.get', array(
'id' => $customer_id
));
// check whether the customer has enough resources to get the subdomain added
if ($customer['subdomains_used'] >= $customer['subdomains'] && $customer['subdomains'] != '-1') {
throw new Exception("Customer has no more resources available", 406);
}
} else {
$customer_id = $this->getUserDetail('customerid');
$customer = $this->getUserData();
}
$alias_stmt = Database::prepare("SELECT COUNT(`id`) AS count FROM `" . TABLE_PANEL_DOMAINS . "` WHERE `aliasdomain`= :aliasdomain");
$alias_check = Database::pexecute_first($alias_stmt, array(
"aliasdomain" => $result['id']
));
$alias_check = $alias_check['count'];
// alias domain checked?
if ($aliasdomain != 0) {
$aliasdomain_stmt = Database::prepare("
SELECT `id` FROM `" . TABLE_PANEL_DOMAINS . "` `d`,`" . TABLE_PANEL_CUSTOMERS . "` `c`
WHERE `d`.`customerid`= :customerid
AND `d`.`aliasdomain` IS NULL
AND `d`.`id`<>`c`.`standardsubdomain`
AND `c`.`customerid`= :customerid
AND `d`.`id`= :id
");
$aliasdomain_check = Database::pexecute_first($aliasdomain_stmt, array(
"id" => $aliasdomain,
"customerid" => $customer_id
), true, true);
if ($aliasdomain_check['id'] != $aliasdomain) {
standard_error('domainisaliasorothercustomer', '', true);
}
triggerLetsEncryptCSRForAliasDestinationDomain($aliasdomain, $this->logger());
}
// validate / correct path/url of domain
$path = $this->validateDomainDocumentRoot($path, $url, $customer, $result['domain'], $_doredirect);
// set alias-fields according to selected alias mode
$iswildcarddomain = ($selectserveralias == '0') ? '1' : '0';
$wwwserveralias = ($selectserveralias == '1') ? '1' : '0';
// if allowed, check for 'is email domain'-flag
if ($result['parentdomainid'] != '0' && ($result['subcanemaildomain'] == '1' || $result['subcanemaildomain'] == '2') && $isemaildomain != $result['isemaildomain']) {
$isemaildomain = intval($isemaildomain);
} else {
$isemaildomain = $result['subcanemaildomain'] == '3' ? 1 : 0;
}
// check changes of openbasedir-path variable
if ($openbasedir_path != 1) {
$openbasedir_path = 0;
}
if ($ssl_redirect != 0) {
// a ssl-redirect only works if there actually is a
// ssl ip/port assigned to the domain
if (domainHasSslIpPort($result['id']) == true) {
$ssl_redirect = '1';
$_doredirect = true;
} else {
standard_error('sslredirectonlypossiblewithsslipport', '', true);
}
}
if ($letsencrypt != 0) {
// let's encrypt only works if there actually is a
// ssl ip/port assigned to the domain
if (domainHasSslIpPort($result['id']) == true) {
$letsencrypt = '1';
} else {
standard_error('letsencryptonlypossiblewithsslipport', '', true);
}
}
// We can't enable let's encrypt for wildcard - domains when using acme-v1
if ($iswildcarddomain == '1' && $letsencrypt == '1' && Settings::Get('system.leapiversion') == '1') {
standard_error('nowildcardwithletsencrypt');
}
// if using acme-v2 we cannot issue wildcard-certificates
// because they currently only support the dns-01 challenge
if ($iswildcarddomain == '0' && $letsencrypt == '1' && Settings::Get('system.leapiversion') == '2') {
standard_error('nowildcardwithletsencryptv2');
}
// Temporarily deactivate ssl_redirect until Let's Encrypt certificate was generated
if ($ssl_redirect > 0 && $letsencrypt == 1 && $result['letsencrypt'] != $letsencrypt) {
$ssl_redirect = 2;
}
// is-email-domain flag changed - remove mail accounts and mail-addresses
if (($result['isemaildomain'] == '1') && $isemaildomain == '0') {
$params = array(
"customerid" => $customer['customerid'],
"domainid" => $id
);
$stmt = Database::prepare("DELETE FROM `" . TABLE_MAIL_USERS . "` WHERE `customerid`= :customerid AND `domainid`= :domainid");
Database::pexecute($stmt, $params, true, true);
$stmt = Database::prepare("DELETE FROM `" . TABLE_MAIL_VIRTUAL . "` WHERE `customerid`= :customerid AND `domainid`= :domainid");
Database::pexecute($stmt, $params, true, true);
$idna_convert = new idna_convert_wrapper();
$this->logger()->logAction($this->isAdmin() ? ADM_ACTION : USR_ACTION, LOG_NOTICE, "[API] automatically deleted mail-table entries for '" . $idna_convert->decode($result['domain']) . "'");
}
// handle redirect
if ($_doredirect) {
updateRedirectOfDomain($id, $redirectcode);
}
if ($path != $result['documentroot'] || $isemaildomain != $result['isemaildomain'] || $wwwserveralias != $result['wwwserveralias'] || $iswildcarddomain != $result['iswildcarddomain'] || $aliasdomain != $result['aliasdomain'] || $openbasedir_path != $result['openbasedir_path'] || $ssl_redirect != $result['ssl_redirect'] || $letsencrypt != $result['letsencrypt'] || $hsts_maxage != $result['hsts'] || $hsts_sub != $result['hsts_sub'] || $hsts_preload != $result['hsts_preload'] || $phpsettingid != $result['phpsettingid']) {
$stmt = Database::prepare("
UPDATE `" . TABLE_PANEL_DOMAINS . "` SET
`documentroot`= :documentroot,
`isemaildomain`= :isemaildomain,
`wwwserveralias`= :wwwserveralias,
`iswildcarddomain`= :iswildcarddomain,
`aliasdomain`= :aliasdomain,
`openbasedir_path`= :openbasedir_path,
`ssl_redirect`= :ssl_redirect,
`letsencrypt`= :letsencrypt,
`hsts` = :hsts,
`hsts_sub` = :hsts_sub,
`hsts_preload` = :hsts_preload,
`phpsettingid` = :phpsettingid
WHERE `customerid`= :customerid AND `id`= :id
");
$params = array(
"documentroot" => $path,
"isemaildomain" => $isemaildomain,
"wwwserveralias" => $wwwserveralias,
"iswildcarddomain" => $iswildcarddomain,
"aliasdomain" => ($aliasdomain != 0 && $alias_check == 0) ? $aliasdomain : null,
"openbasedir_path" => $openbasedir_path,
"ssl_redirect" => $ssl_redirect,
"letsencrypt" => $letsencrypt,
"hsts" => $hsts_maxage,
"hsts_sub" => $hsts_sub,
"hsts_preload" => $hsts_preload,
"phpsettingid" => $phpsettingid,
"customerid" => $customer['customerid'],
"id" => $id
);
Database::pexecute($stmt, $params, true, true);
if ($result['aliasdomain'] != $aliasdomain) {
// trigger when domain id for alias destination has changed: both for old and new destination
triggerLetsEncryptCSRForAliasDestinationDomain($result['aliasdomain'], $this->logger());
triggerLetsEncryptCSRForAliasDestinationDomain($aliasdomain, $this->logger());
} elseif ($result['wwwserveralias'] != $wwwserveralias || $result['letsencrypt'] != $letsencrypt) {
// or when wwwserveralias or letsencrypt was changed
triggerLetsEncryptCSRForAliasDestinationDomain($aliasdomain, $this->logger());
}
// check whether LE has been disabled, so we remove the certificate
if ($letsencrypt == '0' && $result['letsencrypt'] == '1') {
$del_stmt = Database::prepare("
DELETE FROM `" . TABLE_PANEL_DOMAIN_SSL_SETTINGS . "` WHERE `domainid` = :id
");
Database::pexecute($del_stmt, array(
'id' => $id
), true, true);
}
inserttask('1');
inserttask('4');
$idna_convert = new idna_convert_wrapper();
$this->logger()->logAction($this->isAdmin() ? ADM_ACTION : USR_ACTION, LOG_INFO, "[API] edited domain '" . $idna_convert->decode($result['domain']) . "'");
}
$result = $this->apiCall('SubDomains.get', array(
'id' => $id
));
return $this->response(200, "successfull", $result);
} }
public function listing() public function listing()
@@ -503,7 +714,7 @@ class SubDomains extends ApiCommand implements ResourceEntity
if ($this->isAdmin() == false && Settings::IsInList('panel.customer_hide_options', 'domains')) { if ($this->isAdmin() == false && Settings::IsInList('panel.customer_hide_options', 'domains')) {
throw new Exception("You cannot access this resource", 405); throw new Exception("You cannot access this resource", 405);
} }
$result = $this->apiCall('SubDomains.get', array( $result = $this->apiCall('SubDomains.get', array(
'id' => $id, 'id' => $id,
'domainname' => $domainname 'domainname' => $domainname
@@ -600,4 +811,45 @@ class SubDomains extends ApiCommand implements ResourceEntity
$this->logger()->logAction($this->isAdmin() ? ADM_ACTION : USR_ACTION, LOG_WARNING, "[API] deleted subdomain '" . $result['domain'] . "'"); $this->logger()->logAction($this->isAdmin() ? ADM_ACTION : USR_ACTION, LOG_WARNING, "[API] deleted subdomain '" . $result['domain'] . "'");
return $this->response(200, "successfull", $result); return $this->response(200, "successfull", $result);
} }
/**
* validate given path and replace with url if given and valid
*
* @param string $path
* @param string $url
* @param array $customer
* @param string $completedomain
* @param boolean $_doredirect
*
* @return string validated path
*/
private function validateDomainDocumentRoot($path = null, $url = null, $customer = null, $completedomain = null, &$_doredirect = false)
{
// check whether an URL was specified
$_doredirect = false;
if (! empty($url) && validateUrl($url)) {
$path = $url;
$_doredirect = true;
} else {
$path = validate($path, 'path', '', '', array(), true);
}
// check whether path is a real path
if (! preg_match('/^https?\:\/\//', $path) || ! validateUrl($path)) {
if (strstr($path, ":") !== false) {
standard_error('pathmaynotcontaincolon', '', true);
}
// If path is empty or '/' and 'Use domain name as default value for DocumentRoot path' is enabled in settings,
// set default path to subdomain or domain name
if ((($path == '') || ($path == '/')) && Settings::Get('system.documentroot_use_default_value') == 1) {
$path = makeCorrectDir($customer['documentroot'] . '/' . $completedomain);
} else {
$path = makeCorrectDir($customer['documentroot'] . '/' . $path);
}
} else {
// no it's not, create a redirect
$_doredirect = true;
}
return $path;
}
} }

32
lib/functions/validate/function.validateUrl.php
View File

@@ -20,53 +20,49 @@
/** /**
* Returns whether a URL is in a correct format or not * Returns whether a URL is in a correct format or not
* *
* @param string URL to be tested * @param
* string URL to be tested
* @return bool * @return bool
* @author Christian Hoffmann * @author Christian Hoffmann
* @author Froxlor team <team@froxlor.org> (2010-) * @author Froxlor team <team@froxlor.org> (2010-)
* *
*/ */
function validateUrl($url) { function validateUrl($url)
{
global $idna_convert, $theme; if (strtolower(substr($url, 0, 7)) != "http://" && strtolower(substr($url, 0, 8)) != "https://") {
if (strtolower(substr($url, 0, 7)) != "http://"
&& strtolower(substr($url, 0, 8)) != "https://"
) {
$url = 'http://' . $url; $url = 'http://' . $url;
} }
// needs converting // needs converting
try { try {
$idna_convert = new idna_convert_wrapper();
$url = $idna_convert->encode($url); $url = $idna_convert->encode($url);
} catch (Exception $e) { } catch (Exception $e) {
return false; return false;
} }
$pattern = "/^https?:\/\/[a-zA-Z0-9\-\.]+\.[a-zA-Z]{2,4}(\:[0-9]+)?\/?(.+)?$/i"; $pattern = "/^https?:\/\/[a-zA-Z0-9\-\.]+\.[a-zA-Z]{2,4}(\:[0-9]+)?\/?(.+)?$/i";
if (preg_match($pattern, $url)) { if (preg_match($pattern, $url)) {
return true; return true;
} }
// not an fqdn // not an fqdn
if (strtolower(substr($url, 0, 7)) == "http://" if (strtolower(substr($url, 0, 7)) == "http://" || strtolower(substr($url, 0, 8)) == "https://") {
|| strtolower(substr($url, 0, 8)) == "https://"
) {
if (strtolower(substr($url, 0, 7)) == "http://") { if (strtolower(substr($url, 0, 7)) == "http://") {
$ip = strtolower(substr($url, 7)); $ip = strtolower(substr($url, 7));
} }
if (strtolower(substr($url, 0, 8)) == "https://") { if (strtolower(substr($url, 0, 8)) == "https://") {
$ip = strtolower(substr($url, 8)); $ip = strtolower(substr($url, 8));
} }
$ip = substr($ip, 0, strpos($ip, '/')); $ip = substr($ip, 0, strpos($ip, '/'));
// possible : in IP (when a port is given), #1173 // possible : in IP (when a port is given), #1173
// but only if there actually IS ONE // but only if there actually IS ONE
if (strpos($ip, ':') !== false) { if (strpos($ip, ':') !== false) {
$ip = substr($ip, 0, strpos($ip, ':')); $ip = substr($ip, 0, strpos($ip, ':'));
} }
if (validate_ip($ip, true) !== false) { if (validate_ip($ip, true) !== false) {
return true; return true;
} else { } else {

5
tests/Domains/DomainsTest.php
View File

@@ -91,11 +91,14 @@ class DomainsTest extends TestCase
$data = [ $data = [
'domain' => 'test2.local', 'domain' => 'test2.local',
'customerid' => 1, 'customerid' => 1,
'ipandport' => 3 'ipandport' => 3,
'isemaildomain' => 1,
'subcanemaildomain' => 2
]; ];
$json_result = Domains::getLocal($reseller_userdata, $data)->add(); $json_result = Domains::getLocal($reseller_userdata, $data)->add();
$result = json_decode($json_result, true)['data']; $result = json_decode($json_result, true)['data'];
$this->assertEquals('test2.local', $result['domain']); $this->assertEquals('test2.local', $result['domain']);
$this->assertEquals(2, $result['subcanemaildomain']);
} }
public function testAdminDomainsAddSysHostname() public function testAdminDomainsAddSysHostname()

84
tests/SubDomains/SubDomainsTest.php
View File

@@ -8,6 +8,7 @@ use PHPUnit\Framework\TestCase;
*/ */
class SubDomainsTest extends TestCase class SubDomainsTest extends TestCase
{ {
public function testCustomerSubDomainsAdd() public function testCustomerSubDomainsAdd()
{ {
global $admin_userdata; global $admin_userdata;
@@ -46,7 +47,7 @@ class SubDomainsTest extends TestCase
$result = json_decode($json_result, true)['data']; $result = json_decode($json_result, true)['data'];
$this->assertEquals('mysub2.test2.local', $result['domain']); $this->assertEquals('mysub2.test2.local', $result['domain']);
} }
public function testCustomerSubDomainsAddNoPunycode() public function testCustomerSubDomainsAddNoPunycode()
{ {
global $admin_userdata; global $admin_userdata;
@@ -100,7 +101,7 @@ class SubDomainsTest extends TestCase
$this->expectExceptionMessage("Wrong Input in Field 'Domain'"); $this->expectExceptionMessage("Wrong Input in Field 'Domain'");
SubDomains::getLocal($customer_userdata, $data)->add(); SubDomains::getLocal($customer_userdata, $data)->add();
} }
/** /**
* @depends testCustomerSubDomainsAdd * @depends testCustomerSubDomainsAdd
*/ */
@@ -122,7 +123,72 @@ class SubDomainsTest extends TestCase
$this->assertEquals('mysub.test2.local', $result['domain']); $this->assertEquals('mysub.test2.local', $result['domain']);
$this->assertEquals(1, $result['customerid']); $this->assertEquals(1, $result['customerid']);
} }
/**
* @depends testCustomerSubDomainsAdd
*/
public function testAdminSubDomainsGetMainDomain()
{
global $admin_userdata;
// get customer
$json_result = Customers::getLocal($admin_userdata, array(
'loginname' => 'test1'
))->get();
$customer_userdata = json_decode($json_result, true)['data'];
$data = [
'domainname' => 'test2.local'
];
$json_result = SubDomains::getLocal($admin_userdata, $data)->get();
$result = json_decode($json_result, true)['data'];
$this->assertEquals('test2.local', $result['domain']);
$this->assertEquals(1, $result['customerid']);
}
/**
* @depends testCustomerSubDomainsAdd
*/
public function testAdminSubDomainsUpdate()
{
global $admin_userdata;
// get customer
$json_result = Customers::getLocal($admin_userdata, array(
'loginname' => 'test1'
))->get();
$customer_userdata = json_decode($json_result, true)['data'];
$data = [
'domainname' => 'mysub.test2.local',
'path' => 'mysub.test2.local',
'isemaildomain' => 1,
'customer_id' => $customer_userdata['customerid']
];
$json_result = SubDomains::getLocal($admin_userdata, $data)->update();
$result = json_decode($json_result, true)['data'];
$this->assertEquals($customer_userdata['documentroot'] . 'mysub.test2.local/', $result['documentroot']);
}
/**
* @depends testAdminSubDomainsUpdate
*/
public function testCustomerSubDomainsUpdate()
{
global $admin_userdata;
// get customer
$json_result = Customers::getLocal($admin_userdata, array(
'loginname' => 'test1'
))->get();
$customer_userdata = json_decode($json_result, true)['data'];
$data = [
'domainname' => 'mysub.test2.local',
'url' => 'https://www.froxlor.org/',
'isemaildomain' => 0,
];
$json_result = SubDomains::getLocal($customer_userdata, $data)->update();
$result = json_decode($json_result, true)['data'];
$this->assertEquals('https://www.froxlor.org/', $result['documentroot']);
}
public function testCustomerSubDomainsList() public function testCustomerSubDomainsList()
{ {
global $admin_userdata; global $admin_userdata;
@@ -136,7 +202,7 @@ class SubDomainsTest extends TestCase
$result = json_decode($json_result, true)['data']; $result = json_decode($json_result, true)['data'];
$this->assertEquals(3, $result['count']); $this->assertEquals(3, $result['count']);
} }
public function testResellerSubDomainsList() public function testResellerSubDomainsList()
{ {
global $admin_userdata; global $admin_userdata;
@@ -154,11 +220,13 @@ class SubDomainsTest extends TestCase
public function testAdminSubDomainsListWithCustomer() public function testAdminSubDomainsListWithCustomer()
{ {
global $admin_userdata; global $admin_userdata;
$json_result = SubDomains::getLocal($admin_userdata, ['loginname' => 'test1'])->listing(); $json_result = SubDomains::getLocal($admin_userdata, [
'loginname' => 'test1'
])->listing();
$result = json_decode($json_result, true)['data']; $result = json_decode($json_result, true)['data'];
$this->assertEquals(3, $result['count']); $this->assertEquals(3, $result['count']);
} }
/** /**
* @depends testCustomerSubDomainsList * @depends testCustomerSubDomainsList
*/ */
@@ -170,7 +238,9 @@ class SubDomainsTest extends TestCase
'loginname' => 'test1' 'loginname' => 'test1'
))->get(); ))->get();
$customer_userdata = json_decode($json_result, true)['data']; $customer_userdata = json_decode($json_result, true)['data'];
$json_result = SubDomains::getLocal($customer_userdata, ['domainname' => 'mysub.test2.local'])->delete(); $json_result = SubDomains::getLocal($customer_userdata, [
'domainname' => 'mysub.test2.local'
])->delete();
$result = json_decode($json_result, true)['data']; $result = json_decode($json_result, true)['data'];
$this->assertEquals('mysub.test2.local', $result['domain']); $this->assertEquals('mysub.test2.local', $result['domain']);
$this->assertEquals($customer_userdata['customerid'], $result['customerid']); $this->assertEquals($customer_userdata['customerid'], $result['customerid']);