Merge pull request #3420 from ashley-cui/remoteconf

Fix crash for when remote host IP or Username is not set in conf file…
This commit is contained in:
OpenShift Merge Robot
2019-06-26 22:25:34 +02:00
committed by GitHub
2 changed files with 9 additions and 3 deletions

View File

@@ -22,7 +22,7 @@ func ReadRemoteConfig(reader io.Reader) (*RemoteConfig, error) {
// We need to validate each remote connection has fields filled out
for name, conn := range remoteConfig.Connections {
if len(conn.Destination) < 1 {
return nil, errors.Errorf("connection %s has no destination defined", name)
return nil, errors.Errorf("connection %q has no destination defined", name)
}
}
return &remoteConfig, err