Replies: 2 comments
-
|
Related: #4203 |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
Fixed in https://github.com/deployphp/deployer/releases/tag/v8.0.1 |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Deployer Version
v8.0.0
Target OS
Ubuntu 24.04.4
Which PHP version are you using?
PHP 8.5
Content of deploy.php or deploy.yaml
require 'recipe/common.php';
require 'contrib/rsync.php';
import('.hosts.yml');
Steps to reproduce
.hosts.yml
Error
Expected behavior
The rsync task should accept an integer port from a YAML inventory, or cast SSH command arguments to strings before calling string functions.
Workaround
Quoting the port fixes the issue:
Omitting
portalso works when using the default SSH port 22.Possible fix
Cast options to string inside
rsync_rsh()before callingstr_contains()/preg_match().Beta Was this translation helpful? Give feedback.
All reactions