Fix #16954.

Author: dkocher

ssh/src/main/java/ch/cyberduck/core/sftp/openssh/OpenSSHCredentialsConfigurator.java

@@ -53,7 +53,7 @@ public Credentials configure(final Host host) {
             if(!credentials.isPublicKeyAuthentication()) {
                 if(null != entry.getIdentityFile()) {
                     log.info("Using identity {} from {}", entry, configuration);
-                    credentials.setIdentity(entry.getIdentityFile());
+                    credentials.setIdentity(LocalFactory.get(entry.getIdentityFile()));
                 }
                 else {
                     // No custom public key authentication configuration

ssh/src/main/java/ch/cyberduck/core/sftp/openssh/OpenSSHIdentityAgentConfigurator.java

@@ -18,7 +18,6 @@
  * [email protected]
  */
 
-import ch.cyberduck.core.Local;
 import ch.cyberduck.core.LocalFactory;
 import ch.cyberduck.core.sftp.openssh.config.transport.OpenSshConfig;
 
@@ -39,13 +38,13 @@ public OpenSSHIdentityAgentConfigurator(final OpenSshConfig configuration) {
     }
 
     public String getIdentityAgent(final String alias) {
-        final Local agent = configuration.lookup(alias).getIdentityAgent();
+        final String agent = configuration.lookup(alias).getIdentityAgent();
         if(null == agent) {
             log.debug("No configuration for alias {}", alias);
             return null;
         }
         log.debug("Found configuration {} for alias {}", agent, alias);
-        return agent.getAbsolute();
+        return agent;
     }
 
     @Override

ssh/src/main/java/ch/cyberduck/core/sftp/openssh/config/transport/OpenSshConfig.java

@@ -186,7 +186,7 @@ private Map<String, Host> parse(final InputStream in) throws IOException {
             else if("ProxyJump".equalsIgnoreCase(keyword)) {
                 for(final Host c : current) {
                     if(c.proxyJump == null) {
-                        c.proxyJump = dequote(argValue);
+                        c.proxyJump = none(dequote(argValue));
                     }
                 }
             }
@@ -213,21 +213,21 @@ else if("Port".equalsIgnoreCase(keyword)) {
             else if("IdentityFile".equalsIgnoreCase(keyword)) {
                 for(final Host c : current) {
                     if(c.identityFile == null) {
-                        c.identityFile = LocalFactory.get(dequote(argValue));
+                        c.identityFile = none(dequote(argValue));
                     }
                 }
             }
             else if("IdentityAgent".equalsIgnoreCase(keyword)) {
                 for(final Host c : current) {
                     if(c.identityAgent == null) {
-                        c.identityAgent = LocalFactory.get(dequote(argValue));
+                        c.identityAgent = none(dequote(argValue));
                     }
                 }
             }
             else if("PreferredAuthentications".equalsIgnoreCase(keyword)) {
                 for(final Host c : current) {
                     if(c.preferredAuthentications == null) {
-                        c.preferredAuthentications = nows(dequote(argValue));
+                        c.preferredAuthentications = none(nows(dequote(argValue)));
                     }
                 }
             }
@@ -289,6 +289,13 @@ private static Boolean yesno(final String value) {
         return Boolean.FALSE;
     }
 
+    private static String none(final String value) {
+        if("none".equalsIgnoreCase(value)) {
+            return null;
+        }
+        return value;
+    }
+
     /**
      * Configuration of one "Host" block in the configuration file.
      * <p/>
@@ -304,8 +311,8 @@ public static class Host {
         String hostName;
         String proxyJump;
         int port;
-        Local identityFile;
-        Local identityAgent;
+        String identityFile;
+        String identityAgent;
         String user;
         String preferredAuthentications;
         Boolean identitiesOnly;
@@ -363,14 +370,14 @@ public int getPort() {
          * @return path of the private key file to use for authentication; null if the caller should use default
          * authentication strategies.
          */
-        public Local getIdentityFile() {
+        public String getIdentityFile() {
             return identityFile;
         }
 
         /**
          * @return Specifies the UNIX-domain socket used to communicate with the authentication agent.
          */
-        public Local getIdentityAgent() {
+        public String getIdentityAgent() {
             return identityAgent;
         }