package.json

@@ -1,6 +1,6 @@
 {
   "name": "creddy",
-  "version": "0.5.3",
+  "version": "0.5.2",
   "scripts": {
     "dev": "vite",
     "build": "vite build",

src-tauri/Cargo.lock

@@ -1196,7 +1196,7 @@ dependencies = [
 
 [[package]]
 name = "creddy"
-version = "0.5.3"
+version = "0.5.2"
 dependencies = [
  "argon2",
  "auto-launch",

src-tauri/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "creddy"
-version = "0.5.3"
+version = "0.5.2"
 description = "A friendly AWS credentials manager"
 authors = ["Joseph Montanaro"]
 license = ""

src-tauri/src/bin/creddy_cli.rs

@@ -11,7 +11,13 @@ use std::{
 
 
 fn main() {
-    let res = match cli::parser().get_matches().subcommand() {
+    let args = cli::parser().get_matches();
+    if let Some(true) = args.get_one::<bool>("help") {
+        cli::parser().print_help().unwrap(); // if we can't print help we can't print an error
+        process::exit(0);
+    }
+
+    let res = match args.subcommand() {
         None | Some(("run", _)) => launch_gui(),
         Some(("get", m)) => cli::get(m),
         Some(("exec", m)) => cli::exec(m),

src-tauri/src/cli.rs

@@ -1,5 +1,4 @@
 use std::ffi::OsString;
-use std::path::PathBuf;
 use std::process::Command as ChildCommand;
 #[cfg(windows)]
 use std::time::Duration;
@@ -10,7 +9,6 @@ use clap::{
     ArgMatches,
     ArgAction,
     builder::PossibleValuesParser,
-    value_parser,
  };
 use tokio::io::{AsyncReadExt, AsyncWriteExt};
 
@@ -39,14 +37,6 @@ pub fn parser() -> Command<'static> {
     Command::new("creddy")
         .version(env!("CARGO_PKG_VERSION"))
         .about("A friendly AWS credentials manager")
-        .arg(
-            Arg::new("server_addr")
-                .short('a')
-                .long("server-addr")
-                .takes_value(true)
-                .value_parser(value_parser!(PathBuf))
-                .help("Connect to the main Creddy process at this address")
-        )
         .subcommand(
             Command::new("run")
                 .about("Launch Creddy")
@@ -81,7 +71,6 @@ pub fn parser() -> Command<'static> {
                     Arg::new("name")
                         .short('n')
                         .long("name")
-                        .takes_value(true)
                         .help("If unspecified, use default credentials")
                 )
                 .arg(
@@ -105,9 +94,8 @@ pub fn parser() -> Command<'static> {
 pub fn get(args: &ArgMatches) -> Result<(), CliError> {
     let name = args.get_one("name").cloned();
     let base = *args.get_one("base").unwrap_or(&false);
-    let addr = args.get_one("server_addr").cloned();
     
-    let output = match make_request(addr, &Request::GetAwsCredentials { name, base })? {
+    let output = match make_request(&Request::GetAwsCredentials { name, base })? {
         Response::AwsBase(creds) => serde_json::to_string(&creds).unwrap(),
         Response::AwsSession(creds) => serde_json::to_string(&creds).unwrap(),
         r => return Err(RequestError::Unexpected(r).into()),
@@ -120,7 +108,6 @@ pub fn get(args: &ArgMatches) -> Result<(), CliError> {
 pub fn exec(args: &ArgMatches) -> Result<(), CliError> {
     let name = args.get_one("name").cloned();
     let base = *args.get_one("base").unwrap_or(&false);
-    let addr = args.get_one("server_addr").cloned();
     let mut cmd_line = args.get_many("command")
         .ok_or(ExecError::NoCommand)?;
 
@@ -128,7 +115,7 @@ pub fn exec(args: &ArgMatches) -> Result<(), CliError> {
     let mut cmd = ChildCommand::new(cmd_name);
     cmd.args(cmd_line);
     
-    match make_request(addr, &Request::GetAwsCredentials { name, base })? {
+    match make_request(&Request::GetAwsCredentials { name, base })? {
         Response::AwsBase(creds) => {
             cmd.env("AWS_ACCESS_KEY_ID", creds.access_key_id);
             cmd.env("AWS_SECRET_ACCESS_KEY", creds.secret_access_key);
@@ -173,7 +160,6 @@ pub fn exec(args: &ArgMatches) -> Result<(), CliError> {
 
 
 pub fn invoke_shortcut(args: &ArgMatches) -> Result<(), CliError> {
-    let addr = args.get_one("server_addr").cloned();
     let action = match args.get_one::<String>("action").map(|s| s.as_str()) {
         Some("show_window") => ShortcutAction::ShowWindow,
         Some("launch_terminal") => ShortcutAction::LaunchTerminal,
@@ -181,7 +167,7 @@ pub fn invoke_shortcut(args: &ArgMatches) -> Result<(), CliError> {
     };
 
     let req = Request::InvokeShortcut(action);
-    match make_request(addr, &req) {
+    match make_request(&req) {
         Ok(Response::Empty) => Ok(()),
         Ok(r) => Err(RequestError::Unexpected(r).into()),
         Err(e) => Err(e.into()),
@@ -190,12 +176,12 @@ pub fn invoke_shortcut(args: &ArgMatches) -> Result<(), CliError> {
 
 
 #[tokio::main]
-async fn make_request(addr: Option<PathBuf>, req: &Request) -> Result<Response, RequestError> {
+async fn make_request(req: &Request) -> Result<Response, RequestError> {
     let mut data = serde_json::to_string(req).unwrap();
     // server expects newline marking end of request
     data.push('\n');
 
-    let mut stream = connect(addr).await?;
+    let mut stream = connect().await?;
     stream.write_all(&data.as_bytes()).await?;
 
     let mut buf = Vec::with_capacity(1024);
@@ -206,10 +192,10 @@ async fn make_request(addr: Option<PathBuf>, req: &Request) -> Result<Response,
 
 
 #[cfg(windows)]
-async fn connect(addr: Option<PathBuf>) -> Result<NamedPipeClient, std::io::Error> {
+async fn connect() -> Result<NamedPipeClient, std::io::Error> {
     // apparently attempting to connect can fail if there's already a client connected
     loop {
-        let addr = addr.unwrap_or_else(|| srv::addr("creddy-server"));
+        let addr = srv::addr("creddy-server");
         match ClientOptions::new().open(&addr) {
             Ok(stream) => return Ok(stream),
             Err(e) if e.raw_os_error() == Some(ERROR_PIPE_BUSY.0 as i32) => (),
@@ -221,7 +207,7 @@ async fn connect(addr: Option<PathBuf>) -> Result<NamedPipeClient, std::io::Erro
 
 
 #[cfg(unix)]
-async fn connect(addr: Option<PathBuf>) -> Result<UnixStream, std::io::Error> {
+async fn connect() -> Result<UnixStream, std::io::Error> {
-    let path = addr.unwrap_or_else(|| srv::addr("creddy-server"));
+    let path = srv::addr("creddy-server");
     UnixStream::connect(&path).await
 }

src-tauri/src/credentials/aws.rs

@@ -203,6 +203,19 @@ mod tests {
         )
     }
 
+    fn test_uuid() -> Uuid {
+        Uuid::try_parse("00000000-0000-0000-0000-000000000000").unwrap()
+    }
+
+    fn test_uuid_2() -> Uuid {
+        Uuid::try_parse("ffffffff-ffff-ffff-ffff-ffffffffffff").unwrap()
+    }
+
+    fn test_uuid_random() -> Uuid {
+        let bytes = Crypto::salt();
+        Uuid::from_slice(&bytes[..16]).unwrap()
+    }
+
 
     #[sqlx::test(fixtures("aws_credentials"))]
     async fn test_load(pool: SqlitePool) {

src-tauri/src/credentials/record.rs

@@ -112,16 +112,15 @@ impl CredentialRecord {
         Ok(Self::from_parts(row, credential))
     }
 
-    #[cfg(test)]
+    // pub async fn load(id: &Uuid, crypto: &Crypto, pool: &SqlitePool) -> Result<Self, LoadCredentialsError> {
-    pub async fn load(id: &Uuid, crypto: &Crypto, pool: &SqlitePool) -> Result<Self, LoadCredentialsError> {
+    //     let row: CredentialRow = sqlx::query_as("SELECT * FROM credentials WHERE id = ?")
-        let row: CredentialRow = sqlx::query_as("SELECT * FROM credentials WHERE id = ?")
+    //         .bind(id)
-            .bind(id)
+    //         .fetch_optional(pool)
-            .fetch_optional(pool)
+    //         .await?
-            .await?
+    //         .ok_or(LoadCredentialsError::NoCredentials)?;
-            .ok_or(LoadCredentialsError::NoCredentials)?;
 
-        Self::load_credential(row, crypto, pool).await
+    //     Self::load_credential(row, crypto, pool).await
-    }
+    // }
 
     pub async fn load_by_name(name: &str, crypto: &Crypto, pool: &SqlitePool) -> Result<Self, LoadCredentialsError> {
         let row: CredentialRow = sqlx::query_as("SELECT * FROM credentials WHERE name = ?")

src-tauri/src/credentials/ssh.rs

@@ -298,6 +298,7 @@ fn deserialize_algorithm<'de, D>(deserializer: D) -> Result<Algorithm, D::Error>
 #[cfg(test)]
 mod tests {
     use std::fs::{self, File};
+    use ssh_key::Fingerprint;
     use sqlx::types::uuid::uuid;
     use super::*;
 
@@ -446,7 +447,7 @@ mod tests {
     async fn test_load_db(pool: SqlitePool) {
         let crypto = Crypto::fixed();
         let id = uuid!("11111111-1111-1111-1111-111111111111");
-        SshKey::load(&id, &crypto, &pool).await
+        let k = SshKey::load(&id, &crypto, &pool).await
             .expect("Failed to load SSH key from database");
     }
 

src-tauri/src/kv.rs

@@ -44,14 +44,12 @@ pub async fn load_bytes(pool: &SqlitePool, name: &str) -> Result<Option<Vec<u8>>
 }
 
 
-// we don't have a need for this right now, but we will some day
+// pub async fn delete(pool: &SqlitePool, name: &str) -> Result<(), sqlx::Error> {
-#[cfg(test)]
+//     sqlx::query!("DELETE FROM kv WHERE name = ?", name)
-pub async fn delete(pool: &SqlitePool, name: &str) -> Result<(), sqlx::Error> {
+//         .execute(pool)
-    sqlx::query!("DELETE FROM kv WHERE name = ?", name)
+//         .await?;
-        .execute(pool)
+//     Ok(())
-        .await?;
+// }
-    Ok(())
-}
 
 
 pub async fn delete_multi(pool: &SqlitePool, names: &[&str]) -> Result<(), sqlx::Error> {    

src-tauri/tauri.conf.json

@@ -50,7 +50,7 @@
     }
   },
   "productName": "creddy",
-  "version": "0.5.3",
+  "version": "0.5.2",
   "identifier": "creddy",
   "plugins": {},
   "app": {