Treat Empty Passphrase as No Passphrase

src-tauri/Cargo.toml

@@ -23,8 +23,8 @@ env_logger = "0.10.0"
 rand = "0.8.5"
 vt100 = "0.15.1"
 openssl = { version = "0.10.45", features = ["vendored"] }
-russh = { git = "https://github.com/mariotaku/russh", branch = "rsa-hash-override", features = ["openssl"] }
-russh-keys = { git = "https://github.com/mariotaku/russh", branch = "rsa-hash-override", features = ["openssl"] }
+russh = { git = "https://github.com/mariotaku/russh", rev = "42f0e2c", features = ["openssl"] }
+russh-keys = { git = "https://github.com/mariotaku/russh", rev = "42f0e2c", features = ["openssl"] }
 tokio = "1.24.2"
 uuid = { version = "1.2.2", features = ["v1"] }
 reqwest = "0.11.14"

src-tauri/src/device_manager/privkey.rs

@@ -10,6 +10,7 @@ impl PrivateKey {
         passphrase: Option<&str>,
         hash: Option<SignatureHash>,
     ) -> Result<KeyPair, Error> {
+        let passphrase = passphrase.filter(|s| !s.is_empty());
         return match self {
             PrivateKey::Path { name } => {
                 load_secret_key_with_hash(ssh_dir().unwrap().join(name), passphrase.clone(), hash)

src/app/add-device/add-device.component.ts

@@ -106,7 +106,8 @@ export class AddDeviceComponent {
   private async confirmVerificationFailure(device: NewDevice, e: Error): Promise<boolean> {
     const ref = MessageDialogComponent.open(this.modalService, {
       title: 'Verification Failed',
-      message: 'Add this device anyway?',
+      message: `Add ${device.name} anyway?`,
+      error: e,
       positive: 'OK',
       negative: 'Cancel',
     });

src/app/shared/components/message-dialog/message-dialog.component.html

@@ -4,10 +4,15 @@ <h5 class="modal-title">{{title}}</h5>
 <div class="modal-body" [ngSwitch]="messageType">
   <p *ngSwitchCase="'string'">{{message}}</p>
   <ng-template *ngSwitchCase="'component'" #messageComponent></ng-template>
+  <div class="card w-100 p-2" *ngIf="error">
+    <label class="card-title">More info about the error:</label>
+    <pre class="card-body w-100 p-0">{{error.message}}</pre>
+  </div>
 </div>
 <div class="modal-footer">
-  <button type="button" class="btn btn-secondary me-auto" (click)="modal.close(null)" *ngIf="alternative">{{alternative}}</button>
+  <button type="button" class="btn btn-secondary me-auto" (click)="modal.close(null)"
+          *ngIf="alternative">{{alternative}}</button>
   <button type="button" class="btn btn-secondary" (click)="modal.close(false)" *ngIf="negative">{{negative}}</button>
-  <button type="button" class="btn" (click)="modal.close(true)" ngbAutofocus *ngIf="positive"
-    [ngClass]="['btn-' + positiveStyle]">{{positive}}</button>
+  <button type="button" class="btn" (click)="modal.close(true)" autofocus *ngIf="positive"
+          [ngClass]="['btn-' + positiveStyle]">{{positive}}</button>
 </div>

src/app/shared/components/message-dialog/message-dialog.component.ts

@@ -24,6 +24,7 @@ export class MessageDialogComponent implements AfterViewInit, MessageDialogConfi
   alternative?: string;
   positiveStyle?: ButtonStyle = 'primary';
   messageExtras?: Record<string, any>;
+  error?: Error;
 
   @ViewChild('messageComponent', {read: ViewContainerRef})
   messageComponent?: ViewContainerRef;
@@ -83,5 +84,6 @@ export interface MessageDialogConfig {
   negative?: string | null;
   alternative?: string | null;
   positiveStyle?: ButtonStyle;
+  error?: Error;
   messageExtras?: { [keys: string]: any };
 }