From 91e0bad6e9cde8660ff4cc7d63ab56f9b1169d6f Mon Sep 17 00:00:00 2001
From: Git'Fellow <12234510+solracsf@users.noreply.github.com>
Date: Sat, 14 Feb 2026 12:07:32 +0100
Subject: [PATCH] fix(sftp): Handle empty port parameter to allow host-defined
 ports

Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
---
 apps/files_external/lib/Lib/Storage/SFTP.php | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/apps/files_external/lib/Lib/Storage/SFTP.php b/apps/files_external/lib/Lib/Storage/SFTP.php
index e275e5b497b08..6504ea610f345 100644
--- a/apps/files_external/lib/Lib/Storage/SFTP.php
+++ b/apps/files_external/lib/Lib/Storage/SFTP.php
@@ -68,9 +68,12 @@ public function __construct(array $parameters) {
 		Stream::register();
 
 		$parsedHost = $this->splitHost($parameters['host']);
-
 		$this->host = $parsedHost[0];
-		$this->port = $parameters['port'] ?? $parsedHost[1];
+
+		// Handle empty port parameter to allow host-defined ports
+		// and ensure strictly numeric ports
+		$parsedPort = $parameters['port'] ?? null;
+		$this->port = (int)(is_numeric($parsedPort) ? $parsedPort : $parsedHost[1]);
 
 		if (!isset($parameters['user'])) {
 			throw new \UnexpectedValueException('no authentication parameters specified');