upstream/nextcloud
1
0
Fork 0
mirror of https://github.com/nextcloud/server.git synced 2026-05-28 04:32:30 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 54

fix(sftp): Handle empty port parameter to allow host-defined ports

Browse source 
Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
This commit is contained in:
Git'Fellow 2026-02-14 12:07:32 +01:00 committed by backportbot[bot]
parent ed055d6a7a
commit f7c97e6c9f
1 changed files with 5 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
apps/files_external/lib/Lib/Storage/SFTP.php
View file

@ -62,9 +62,12 @@ class SFTP extends Common {
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');