Skip to content
This repository has been archived by the owner on Jun 15, 2022. It is now read-only.

Thellimist/mergecpaneldomainbug #97

Merged
merged 3 commits into from
Jun 29, 2016
Merged

Thellimist/mergecpaneldomainbug #97

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
548aab4
fixed bug: module showing inactive even though it's active with Cloud…
thellimist Jun 29, 2016
8dd6302
added, upgraded tests for mergeCpanelAndCFDomains callAPI not getting…
thellimist Jun 29, 2016
14c2064
formating fixes
thellimist Jun 29, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
124 changes: 69 additions & 55 deletions src/Cpanel/ClientActions.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@

namespace CF\Cpanel;

use \CF\API\APIInterface;
use CF\API\APIInterface;
use CF\API\Request;
use \CF\Cpanel\Zone\Partial;
use CF\Cpanel\Zone\Partial;

class ClientActions
{
Expand All @@ -18,8 +18,8 @@ class ClientActions

/**
* @param CpanelIntegration $cpanelIntegration
* @param APIInterface $api
* @param Request $request
* @param APIInterface $api
* @param Request $request
*/
public function __construct(CpanelIntegration $cpanelIntegration, APIInterface $api, Request $request)
{
Expand All @@ -35,122 +35,136 @@ public function __construct(CpanelIntegration $cpanelIntegration, APIInterface $
/**
* @param Partial $partialZoneSet
*/
public function setPartialZoneSet(Partial $partialZoneSet) {
public function setPartialZoneSet(Partial $partialZoneSet)
{
$this->partialZoneSet = $partialZoneSet;
}

/**
* GET /zones
* GET /zones.
*
* @return mixed
*/
public function mergeCpanelAndCFDomains()
{
$cf_zones_list = $this->api->callAPI($this->request);
$get_cpanel_domains = $this->cpanelAPI->getDomainList();

if ($this->api->responseOk($cf_zones_list)) {
$get_cpanel_domains = $this->cpanelAPI->getDomainList();
//addon and primary domains are A records to the main_domain so for Cpanel its always a list of one
$cpanel_domain_list = array($get_cpanel_domains['main_domain']);
if (!is_null($get_cpanel_domains['addon_domains'])) {
$cpanel_domain_list = array_merge($cpanel_domain_list, $get_cpanel_domains['addon_domains']);
}

//addon and primary domains are A records to the main_domain so for Cpanel its always a list of one
$cpanel_domain_list = array($get_cpanel_domains["main_domain"]);
if (!is_null($get_cpanel_domains["addon_domains"])) {
$cpanel_domain_list = array_merge($cpanel_domain_list, $get_cpanel_domains["addon_domains"]);
}
if (!is_null($get_cpanel_domains['parked_domains'])) {
$cpanel_domain_list = array_merge($cpanel_domain_list, $get_cpanel_domains['parked_domains']);
}

if (!is_null($get_cpanel_domains["parked_domains"])) {
$cpanel_domain_list = array_merge($cpanel_domain_list, $get_cpanel_domains["parked_domains"]);
}
$merged_domain_list = array();
foreach ($cpanel_domain_list as $cpanel_domain) {
$found = false;

$request = new Request('GET', 'zones/', array('name' => $cpanel_domain), array());
$cpanel_zone = $this->api->callAPI($request);

$merged_domain_list = array();
foreach ($cpanel_domain_list as $cpanel_domain) {
$found = false;
foreach ($cf_zones_list["result"] as $cf_zone) {
if ($cf_zone["name"] === $cpanel_domain) {
if ($this->api->responseOk($cpanel_zone)) {
foreach ($cpanel_zone['result'] as $cf_zone) {
if ($cf_zone['name'] === $cpanel_domain) {
$found = true;
array_push($merged_domain_list, $cf_zone);
}
}
if ($found === false) {
array_push($merged_domain_list, array(
}

if ($found === false) {
array_push($merged_domain_list, array(
'name' => $cpanel_domain,
'plan' => array('name' => ''),
'type' => '',
'status' => 'inactive'
'status' => 'inactive',
));
}
}
}

$cf_zones_list["result"] = $merged_domain_list;
$cf_zones_list = array();
$cf_zones_list['result'] = $merged_domain_list;
$cf_zones_list['success'] = true;

return $cf_zones_list;
}

/**
* POST /zones/:id/dns_records
* POST /zones/:id/dns_records.
*
* @return array|string
*/
public function createDNSRecord()
{
$create_dns_record_result = $this->api->callAPI($this->request);
if ($this->api->responseOk($create_dns_record_result)) {
$domain_name = $create_dns_record_result["result"]["zone_name"];
$domain_name = $create_dns_record_result['result']['zone_name'];
/*
* Only create the DNS record on CPanel if it was provisioned with CNAME setup, aka if the resolve-to DNS record is set.
*/
if ($this->partialZoneSet->getResolveToDNSRecord($this->cpanelAPI->getDNSRecords($domain_name)) !== null) {
//cpanel record names have trailing dot
$create_dns_record_result["result"]["name"] = $this->cpanelAPI->get_cpanel_dns_record_name($create_dns_record_result["result"]["name"]);
$sub_domain = $create_dns_record_result["result"]["name"];
$create_dns_record_result['result']['name'] = $this->cpanelAPI->get_cpanel_dns_record_name($create_dns_record_result['result']['name']);
$sub_domain = $create_dns_record_result['result']['name'];

if ($this->partialZoneSet->partialZoneSet($sub_domain, $domain_name)) {
return $create_dns_record_result;
} else {
return $this->api->createAPIError("Could not modify Cpanel DNS record for '" . $sub_domain . "', please contact your host.");
return $this->api->createAPIError("Could not modify Cpanel DNS record for '".$sub_domain."', please contact your host.");
}
}
}

return $create_dns_record_result;
}

/**
* PATCH /zones/:id/dns_records/:id
* PATCH /zones/:id/dns_records/:id.
*
* @return string
*/
public function patchDNSRecord()
{
$patch_dns_record_result = $this->api->callAPI($this->request);
if ($this->api->responseOk($patch_dns_record_result)) {
$patch_dns_record_result["result"]["name"] = $this->cpanelAPI->get_cpanel_dns_record_name($patch_dns_record_result["result"]["name"]);
$patch_dns_record_result['result']['name'] = $this->cpanelAPI->get_cpanel_dns_record_name($patch_dns_record_result['result']['name']);
}

return $patch_dns_record_result;
}

/**
* DELETE /zones/:id
* DELETE /zones/:id.
*
* @return mixed
*/
public function deleteZone()
{
$path_array = explode("/", $this->request->getUrl());
$path_array = explode('/', $this->request->getUrl());
$zone_tag = $path_array[1];

//try to remove partial zone set up IF it exists
if ($zone_tag !== null) {
$zone_get_details_response = $this->api->zoneGetDetails($zone_tag);
if ($this->api->responseOk($zone_get_details_response)) {
$domain_name = $zone_get_details_response["result"]["name"];
$domain_name = $zone_get_details_response['result']['name'];
if ($domain_name !== null) {
if ($this->partialZoneSet->removePartialZoneSet($domain_name) === false) {
return $this->api->createAPIError("CPanel was unable to update the DNS records for '" . $domain_name . "' to no longer point at CloudFlare. Please contact your host.");
return $this->api->createAPIError("CPanel was unable to update the DNS records for '".$domain_name."' to no longer point at CloudFlare. Please contact your host.");
}
}
}
}

return $this->api->callAPI($this->request);
}


/**
* GET /zones/:id/dns_records
* GET /zones/:id/dns_records.
*
* @return string
*/
public function mergeDNSRecords()
Expand All @@ -160,46 +174,46 @@ public function mergeDNSRecords()
return $cf_dns_record_list;
}

$path_array = explode("/", $this->request->getUrl());
$path_array = explode('/', $this->request->getUrl());
$zone_id = $path_array[1];

$zone_get_details_response = $this->api->zoneGetDetails($zone_id);
if (!$this->api->responseOk($zone_get_details_response)) {
return $this->api->createAPIError("Could not merge DNS record because zone details could not be retrieved.");
return $this->api->createAPIError('Could not merge DNS record because zone details could not be retrieved.');
}

$domain_name = $zone_get_details_response["result"]["name"];
$domain_name = $zone_get_details_response['result']['name'];

$cpanel_dns_record_list = $this->cpanelAPI->getDNSRecords($domain_name);

if (!$this->cpanelAPI->uapi_response_ok($cf_dns_record_list)) {
return $this->api->createAPIError("Error getting the DNS records for '" . $domain_name . "' from Cpanel.");
return $this->api->createAPIError("Error getting the DNS records for '".$domain_name."' from Cpanel.");
}

//The user cant provision the root domain or the resolve to record so we add them to the name list to prevent them
//from being added when we loop through the cpanel DNS records
$cf_dns_record_name_list = array($domain_name . ".", $this->partialZoneSet->getResolveToValue($domain_name));
$cf_dns_record_name_list = array($domain_name.'.', $this->partialZoneSet->getResolveToValue($domain_name));

foreach ($cf_dns_record_list["result"] as $key => $cf_dns_record) {
foreach ($cf_dns_record_list['result'] as $key => $cf_dns_record) {
//add trailing dot to cf dns record names
$cf_dns_record_list["result"][$key]["name"] = $this->cpanelAPI->get_cpanel_dns_record_name($cf_dns_record_list["result"][$key]["name"]);
$cf_dns_record_list['result'][$key]['name'] = $this->cpanelAPI->get_cpanel_dns_record_name($cf_dns_record_list['result'][$key]['name']);

//build list of dns record names
array_push($cf_dns_record_name_list, $cf_dns_record_list["result"][$key]["name"]);
array_push($cf_dns_record_name_list, $cf_dns_record_list['result'][$key]['name']);
}

foreach ($cpanel_dns_record_list as $cpanel_dns_record) {
//if the record a type CloudFlare can proxy
if (!in_array(strtoupper($cpanel_dns_record->getType()), CpanelDNSRecord::$DNS_RECORDS_CF_CANNOT_PROXY)) {
//if the cpanel record isn't in the array, add it.
if (!in_array($cpanel_dns_record->getName(), $cf_dns_record_name_list)) {
array_push($cf_dns_record_list["result"], array(
"name" => $cpanel_dns_record->getName(),
"type" => $cpanel_dns_record->getType(),
"content" => $cpanel_dns_record->getContent(),
"ttl" => $cpanel_dns_record->getTtl(),
"proxied" => false,
"zone_id" => $zone_id
array_push($cf_dns_record_list['result'], array(
'name' => $cpanel_dns_record->getName(),
'type' => $cpanel_dns_record->getType(),
'content' => $cpanel_dns_record->getContent(),
'ttl' => $cpanel_dns_record->getTtl(),
'proxied' => false,
'zone_id' => $zone_id,
));
}
}
Expand Down
Loading