chore: improve idempotent-proxy-canister API

Cargo.toml

@@ -15,7 +15,7 @@ strip = true
 opt-level = 's'
 
 [workspace.package]
-version = "1.0.0"
+version = "1.0.3"
 edition = "2021"
 repository = "https://github.com/ldclabs/idempotent-proxy"
 keywords = ["idempotent", "reverse", "proxy", "icp"]

examples/eth-canister-lite/dfx.json

@@ -4,6 +4,16 @@
       "candid": "eth-canister-lite.did",
       "package": "eth-canister-lite",
       "type": "rust"
+    },
+    "idempotent-proxy-canister": {
+      "candid": "https://github.com/ldclabs/idempotent-proxy/releases/download/v1.0.3/idempotent_proxy_canister.did",
+      "type": "custom",
+      "wasm": "https://github.com/ldclabs/idempotent-proxy/releases/download/v1.0.3/idempotent_proxy_canister.wasm.gz",
+      "metadata": [
+        {
+          "name": "candid:service"
+        }
+      ]
     }
   },
   "defaults": {

src/idempotent-proxy-canister/idempotent-proxy-canister.did

@@ -44,11 +44,12 @@ type UpgradeArgs = record { proxy_token_refresh_interval : opt nat64 };
 service : (opt ChainArgs) -> {
   admin_add_caller : (principal) -> (Result);
   admin_remove_caller : (principal) -> (Result);
-  admin_set_agent : (vec Agent) -> (Result_1);
+  admin_set_agents : (vec Agent) -> (Result_1);
   admin_set_managers : (vec principal) -> (Result_1);
   get_state : () -> (Result_2) query;
   parallel_call_all_ok : (CanisterHttpRequestArgument) -> (HttpResponse);
   parallel_call_any_ok : (CanisterHttpRequestArgument) -> (HttpResponse);
   proxy_http_request : (CanisterHttpRequestArgument) -> (HttpResponse);
+  validate_admin_set_agents : (vec Agent) -> (Result_1) query;
   validate_admin_set_managers : (vec principal) -> (Result_1) query;
 }

src/idempotent-proxy-canister/src/lib.rs

@@ -16,6 +16,7 @@ static ANONYMOUS: Principal = Principal::anonymous();
 
 #[ic_cdk::update(guard = "is_controller")]
 fn admin_set_managers(args: BTreeSet<Principal>) -> Result<(), String> {
+    validate_admin_set_managers(args.clone())?;
     store::state::with_mut(|r| {
         r.managers = args;
     });
@@ -34,17 +35,24 @@ fn validate_admin_set_managers(args: BTreeSet<Principal>) -> Result<(), String>
 }
 
 #[ic_cdk::update(guard = "is_controller_or_manager")]
-async fn admin_set_agent(agents: Vec<agent::Agent>) -> Result<(), String> {
-    if agents.is_empty() {
-        return Err("agents cannot be empty".to_string());
-    }
+async fn admin_set_agents(agents: Vec<agent::Agent>) -> Result<(), String> {
+    validate_admin_set_agents(agents.clone())?;
 
     let (ecdsa_key_name, proxy_token_refresh_interval) =
         store::state::with(|s| (s.ecdsa_key_name.clone(), s.proxy_token_refresh_interval));
     tasks::update_proxy_token(ecdsa_key_name, proxy_token_refresh_interval, agents).await;
     Ok(())
 }
 
+#[ic_cdk::query]
+fn validate_admin_set_agents(agents: Vec<agent::Agent>) -> Result<(), String> {
+    if agents.is_empty() {
+        return Err("agents cannot be empty".to_string());
+    }
+
+    Ok(())
+}
+
 #[ic_cdk::update(guard = "is_controller_or_manager")]
 fn admin_add_caller(id: Principal) -> Result<bool, String> {
     store::state::with_mut(|r| Ok(r.allowed_callers.insert(id)))