Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NativeSsh, improve user experience for connection failures #1012

Closed
pniederlag opened this issue Feb 14, 2017 · 1 comment
Closed

NativeSsh, improve user experience for connection failures #1012

pniederlag opened this issue Feb 14, 2017 · 1 comment

Comments

@pniederlag
Copy link
Contributor

Q A
Issue Type Feature Request
Deployer Version 4.2.1
Local Machine OS N/A
Remote Machine OS N/A

Description

As suggested I switched to the native ssh connection via set('ssh_type', 'native')

When I do have an error in the setup that prevents a succesfull (ssh-)connection the error shown for the native-ssh implementation is very irritating.

Steps to reproduce

  • add some invalid server configuration
  • try to run an arbitrary task on this server
  • EXCEPTION [RuntimeException]
    Unable to login with the provided credentials is clear to me.
  • switch to native ssh implementation as the warning suggests
  • EXCEPTION: [Symfony\Component\Process\Exception\ProcessFailedException] is irritating for me

Content of deploy.php

// compat with php < 5.6
namespace Deployer;

use Symfony\Component\Console\Input\InputArgument;

//
require 'recipes/rsync.php';

argument('stage', InputArgument::OPTIONAL, 'Run tasks only on this server or group of servers');

//set('ssh_type', 'native');
//set('ssh_multiplexing', true);

// Configuration
serverList('servers.yml');

desc('Say Hello');
task('say_hello', function () {
     writeln('Hello');
     run("pwd");
});%  
pluseg pushed a commit to pluseg/deployer that referenced this issue Feb 24, 2017
pluseg pushed a commit to pluseg/deployer that referenced this issue Feb 24, 2017
antonmedv added a commit that referenced this issue Feb 25, 2017
Change output of errors for native ssh (#1012)
@antonmedv
Copy link
Member

Done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants