rosslh/coolify
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(domains): allow adding domains without explicit ports (#11442)

Browse source
This commit is contained in:
Andras Bacsai 2026-08-21 12:04:14 +02:00 committed by GitHub
parent 15072b9cf7
commit 83f1a2e503
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
app/Support/DomainUrlParts.php
View file

@ -4,11 +4,11 @@
class DomainUrlParts
{
public static function compose(string $scheme, string $host, string $port = '', string $path = ''): string
public static function compose(string $scheme, string $host, ?string $port = '', string $path = ''): string
{
$scheme = strtolower(trim($scheme)) === 'http' ? 'http' : 'https';
$host = trim($host);
$port = trim($port);
$port = trim((string) $port);
$path = trim($path);
if ($path !== '' && ! str_starts_with($path, '/') && ! str_starts_with($path, '?') && ! str_starts_with($path, '#')) {

5
tests/Unit/DomainUrlPartsTest.php
View file

@ -28,3 +28,8 @@
it('preserves an explicitly configured default port', function () {
expect(DomainUrlParts::split('https://app.example.com:443')['port'])->toBe('443');
});
it('composes a domain when Livewire hydrates an empty numeric port as null', function () {
expect(DomainUrlParts::compose('https', 'app.example.com', null))
->toBe('https://app.example.com');
});