chore(sns): Add swap logs when changing controllership (#2027)

This is to aid debugging in situations such as this one: https://forum.dfinity.org/t/dapp-canisters-control-not-returned/36125/5
dfinity · Oct 14, 2024 · e1e0971 · e1e0971
1 parent f1564ab
commit e1e0971
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/rs/sns/swap/src/swap.rs b/rs/sns/swap/src/swap.rs
@@ -1413,7 +1413,10 @@ impl Swap {
         let result = self.restore_dapp_controllers(sns_root_client).await;
 
         match result {
-            Ok(result) => result.into(),
+            Ok(result) => {
+                log!(INFO, "Successfully restored dapp controllers, {:?}", result);
+                SetDappControllersCallResult::from(result)
+            }
             Err(err_message) => {
                 log!(ERROR, "Halting set_dapp_controllers(), {:?}", err_message);
                 SetDappControllersCallResult { possibility: None }
@@ -1431,7 +1434,14 @@ impl Swap {
             .await;
 
         match result {
-            Ok(result) => result.into(),
+            Ok(result) => {
+                log!(
+                    INFO,
+                    "Successfully took sole control of dapp controllers, {:?}",
+                    result
+                );
+                SetDappControllersCallResult::from(result)
+            }
             Err(err_message) => {
                 log!(ERROR, "Halting set_dapp_controllers(), {:?}", err_message);
                 SetDappControllersCallResult { possibility: None }