Fix domain input whitespace trimming in instance settings
- Add trim() to FQDN in instantSave() method to prevent whitespace from being saved - Add trim() to FQDN in submit() method before validation and DNS checks - Prevents invalid HostSNI rules caused by leading/trailing whitespace - Fixes issue where accidental whitespace from copy-paste causes deployment failures Fixes #7797 Co-authored-by: Andras Bacsai <andrasbacsai@users.noreply.github.com>
This commit is contained in:
parent
f488bd9a32
commit
d130030ad6
1 changed files with 6 additions and 1 deletions
|
|
@ -80,7 +80,7 @@ public function timezones(): array
|
|||
public function instantSave($isSave = true)
|
||||
{
|
||||
$this->validate();
|
||||
$this->settings->fqdn = $this->fqdn;
|
||||
$this->settings->fqdn = $this->fqdn ? trim($this->fqdn) : $this->fqdn;
|
||||
$this->settings->public_port_min = $this->public_port_min;
|
||||
$this->settings->public_port_max = $this->public_port_max;
|
||||
$this->settings->instance_name = $this->instance_name;
|
||||
|
|
@ -121,6 +121,11 @@ public function submit()
|
|||
}
|
||||
$this->validate();
|
||||
|
||||
// Trim FQDN to remove leading/trailing whitespace
|
||||
if ($this->fqdn) {
|
||||
$this->fqdn = trim($this->fqdn);
|
||||
}
|
||||
|
||||
if ($this->settings->is_dns_validation_enabled && $this->fqdn) {
|
||||
if (! validateDNSEntry($this->fqdn, $this->server)) {
|
||||
$this->dispatch('error', "Validating DNS failed.<br><br>Make sure you have added the DNS records correctly.<br><br>{$this->fqdn}->{$this->server->ip}<br><br>Check this <a target='_blank' class='underline dark:text-white' href='https://coolify.io/docs/knowledge-base/dns-configuration'>documentation</a> for further help.");
|
||||
|
|
|
|||
Loading…
Reference in a new issue