feat: add parameter guards for all methods (#34)

* feat: add new error type for invalid method arguments

* feat: add parameter guards for the constructor and all methods

* fix: use the correct error type for the auth methods

Author: ctran88

src/auth.rs

@@ -47,6 +47,18 @@ impl Auth {
         external_id: String,
         passkey_display_name: String,
     ) -> Result<String, Error> {
+        if external_id.is_empty() {
+            return Err(Error::InvalidArgument(
+                "external_id is required".to_string(),
+            ));
+        }
+
+        if passkey_display_name.is_empty() {
+            return Err(Error::InvalidArgument(
+                "passkey_display_name is required".to_string(),
+            ));
+        }
+
         transactions_api::create_register_transaction(
             &self.configuration,
             crate::openapi::models::CreateTransactionRegisterRequest {
@@ -91,6 +103,12 @@ impl Auth {
         &self,
         external_id: String,
     ) -> Result<String, Error> {
+        if external_id.is_empty() {
+            return Err(Error::InvalidArgument(
+                "external_id is required".to_string(),
+            ));
+        }
+
         transactions_api::create_authenticate_transaction(
             &self.configuration,
             crate::openapi::models::CreateTransactionAuthenticateRequest { external_id },
@@ -130,6 +148,10 @@ impl Auth {
     /// }
     /// ```
     pub async fn verify_nonce(&self, nonce: String) -> Result<String, Error> {
+        if nonce.is_empty() {
+            return Err(Error::InvalidArgument("nonce is required".to_string()));
+        }
+
         authenticate_api::authenticate_verify_nonce(
             &self.configuration,
             crate::openapi::models::Nonce { nonce },

src/lib.rs

@@ -31,6 +31,7 @@ pub enum Error {
     UserHasNoPasskeys,
     InternalServerError,
     Other(String),
+    InvalidArgument(String),
 }
 
 impl fmt::Display for Error {
@@ -48,6 +49,7 @@ impl fmt::Display for Error {
             Error::UserHasNoPasskeys => ("response", "user has no passkeys".to_string()),
             Error::InternalServerError => ("response", "internal server error".to_string()),
             Error::Other(e) => ("response", e.to_string()),
+            Error::InvalidArgument(e) => ("argument", e.to_string()),
         };
         write!(f, "error in {}: {}", module, e)
     }

src/passage_flex.rs

@@ -29,6 +29,14 @@ impl PassageFlex {
     /// );
     /// ```
     pub fn new(app_id: String, api_key: String) -> Self {
+        if app_id.is_empty() {
+            panic!("A Passage App ID is required. Please include (app_id: YOUR_APP_ID, api_key: YOUR_APP_ID).");
+        }
+
+        if api_key.is_empty() {
+            panic!("A Passage API key is required. Please include (app_id: YOUR_APP_ID, api_key: YOUR_APP_ID).");
+        }
+
         let mut headers = reqwest::header::HeaderMap::with_capacity(1);
         headers.insert(
             "Passage-Version",

src/user.rs

@@ -15,6 +15,12 @@ impl User {
 
     /// Get a user's ID in Passage by their external ID
     async fn get_id(&self, external_id: String) -> Result<String, Error> {
+        if external_id.is_empty() {
+            return Err(Error::InvalidArgument(
+                "external_id is required".to_string(),
+            ));
+        }
+
         let users = users_api::list_paginated_users(
             &self.configuration,
             Some(1),
@@ -172,6 +178,10 @@ impl User {
     /// }
     /// ```
     pub async fn revoke_device(&self, external_id: String, device_id: String) -> Result<(), Error> {
+        if device_id.is_empty() {
+            return Err(Error::InvalidArgument("device_id is required".to_string()));
+        }
+
         let user_id = self.get_id(external_id).await?;
         user_devices_api::delete_user_devices(&self.configuration, &user_id, &device_id)
             .await