Removed caching of the POST response from submitting a confirmation (#…

…846) (#847)

* Removed caching of the POST response from submitting a confirmation

* Version bump to v3.20.1

* Added comment clarifying change in methods

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "safe-client-gateway"
-version = "3.20.0"
+version = "3.20.1"
 authors = ["jpalvarezl <[email protected]>", "rmeissner <[email protected]>", "fmrsabino <[email protected]>"]
 edition = "2018"
 

src/routes/transactions/handlers/proposal.rs

@@ -1,19 +1,18 @@
 use crate::cache::cache_operations::{Invalidate, InvalidationPattern, InvalidationScope};
 use crate::providers::info::{DefaultInfoProvider, InfoProvider};
-use crate::routes::transactions::handlers::details::get_multisig_transaction_details;
-use crate::routes::transactions::models::details::TransactionDetails;
 use crate::routes::transactions::models::requests::MultisigTransactionRequest;
 use crate::utils::context::RequestContext;
 use crate::utils::errors::ApiResult;
 use crate::utils::http_client::Request;
 use serde_json::json;
 
+// The transaction details are returned at the route level. The same as for transaction submission
 pub async fn submit_confirmation(
     context: &RequestContext,
     chain_id: &str,
     safe_tx_hash: &str,
     signature: &str,
-) -> ApiResult<TransactionDetails> {
+) -> ApiResult<()> {
     let info_provider = DefaultInfoProvider::new(chain_id, context);
     let url = core_uri!(
         info_provider,
@@ -36,7 +35,7 @@ pub async fn submit_confirmation(
     .execute()
     .await;
 
-    get_multisig_transaction_details(&info_provider, chain_id, safe_tx_hash).await
+    Ok(())
 }
 
 pub async fn propose_transaction(

src/routes/transactions/routes.rs

@@ -66,21 +66,22 @@ pub async fn post_confirmation<'e>(
     safe_tx_hash: String,
     tx_confirmation_request: Result<Json<ConfirmationRequest>, Error<'e>>,
 ) -> ApiResult<content::Json<String>> {
-    CacheResponse::new(&context)
-        .resp_generator(|| {
-            proposal::submit_confirmation(
-                &context,
-                &chain_id,
-                &safe_tx_hash,
-                &tx_confirmation_request
-                    .as_ref()
-                    .unwrap()
-                    .0
-                    .signed_safe_tx_hash,
-            )
-        })
+    let request: ConfirmationRequest = tx_confirmation_request?.0;
+
+    proposal::submit_confirmation(
+        &context,
+        &chain_id,
+        &safe_tx_hash,
+        &request.signed_safe_tx_hash,
+    )
+    .await?;
+
+    let tx_details = CacheResponse::new(&context)
+        .resp_generator(|| details::get_transactions_details(&context, &chain_id, &safe_tx_hash))
         .execute()
-        .await
+        .await;
+
+    return tx_details;
 }
 
 /// `/v1/chains/<chain_id>/safes/<safe_address>/transactions/history?<cursor>&<timezone_offset>&<trusted>` <br />