chore(api): retry put comments requests (#297)

* chore(api): retry put comments requests
reinfer · Aug 5, 2024 · 255c5bf · 255c5bf
1 parent 9afc496
commit 255c5bf
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,4 +1,5 @@
 # Unreleased
+- Retry when putting comments
 - Add ability to get email by id 
 - Add ability to upload attachment content for comments
 

diff --git a/api/src/lib.rs b/api/src/lib.rs
@@ -522,12 +522,15 @@ impl Client {
         comments: Vec<NewComment>,
         no_charge: bool,
     ) -> Result<SplitableRequestResponse<PutCommentsResponse>> {
+        // Retrying here despite the potential for 409's in order to increase reliability when
+        // working with poor connection
+
         self.splitable_request(
             Method::PUT,
             self.endpoints.put_comments(source_name)?,
             PutCommentsRequest { comments },
             Some(NoChargeQuery { no_charge }),
-            Retry::No,
+            Retry::Yes,
         )
     }
 
@@ -537,12 +540,14 @@ impl Client {
         comments: Vec<NewComment>,
         no_charge: bool,
     ) -> Result<PutCommentsResponse> {
+        // Retrying here despite the potential for 409's in order to increase reliability when
+        // working with poor connection
         self.request(
             &Method::PUT,
             &self.endpoints.put_comments(source_name)?,
             &Some(PutCommentsRequest { comments }),
             &Some(NoChargeQuery { no_charge }),
-            &Retry::No,
+            &Retry::Yes,
         )
     }