Details
Bug
Major
Description
To reproduce:
- Add a domain where the TLD is using OpenSRS as an admin under the "Domains" widget for a client profile.
- Check the logs and note that the name servers set for the TLD and during creation are not included in the API call.
Not sure if ordering as a client has the same issue. Note that no errors are presented with our OpenSRS sandbox, but the name servers specified are not the ones set in the sandbox. It may be that the sandbox does not require name servers because its a sandbox, it may be a sandbox limitation.
In at least some production environments the domain cannot be provisioned, it returns the following error from OpenSRS "Minimum of 2 nameservers must be supplied."
From a customer:
Looking at the API docs, the call seems to be correctly saying “custom_nameservers = 1” .. but then doesn’t include the “nameserver_list” which is required https://domains.opensrs.guide/docs/sw_register-domain-or-trust_service-
The module seems to make 2 calls, one to register the domain and then a second to set the name servers …. Which isn’t what the API would seem to expect pensrs.php:428
// Register domain
$this->registerDomain($vars['domain'], $package->module_row, $fields);
if ($this->Input->errors()) {
return;
}
// Set nameservers
$this->setDomainNameservers($vars['domain'], $package->module_row, [
$fields['nameserver_list'][0]['name'] ?? '',
$fields['nameserver_list'][1]['name'] ?? '',
$fields['nameserver_list'][2]['name'] ?? '',
$fields['nameserver_list'][3]['name'] ?? '',
]);
// Ignore nameserver errors
$this->Input->setErrors([]);
