Saves passphrase for private key in keychain regardless if login fails

[cyberduck]

1bb2969 created the issue

When using key-based authentication to establish an SFTP connection, if the user enters his or her passphrase incorrectly and checks the box to save the passphrase in the system keychain, the passphrase is saved even though the connection is unsuccessful. This results in an incorrect passphrase being stored in the keychain, which can cause confusion for the user. Cyberduck should only store the passphrase if the connection attempt succeeds, as is standard behavior for a modern web browser.

[cyberduck]

@dkocher commented

In d47ced2.