DRIVERS-1641: pre-4.4 mongos writeConcernError does not determine ret…

…ryability Also clarify that pre-4.4 writeErrors[].code should never be used to determine retryability. This is intentionally untested because failCommand does not provide a way to mock writeErrors.
mongodb · Jan 12, 2024 · 87f2617 · 87f2617
1 parent 2176bba
commit 87f2617
Show file tree

Hide file tree

Showing 3 changed files with 422 additions and 3 deletions.
diff --git a/source/retryable-writes/retryable-writes.rst b/source/retryable-writes/retryable-writes.rst
@@ -252,7 +252,8 @@ The RetryableWriteError label might be added to an error in a variety of ways:
   errors that meet the following criteria if the retryWrites option is set to
   true on the client performing the relevant operation:
 
-  - a server error response with any the following codes:
+  - a mongod or mongos response with any the following error codes in the
+    top-level ``code`` field:
 
     .. list-table::
       :header-rows: 1
@@ -284,8 +285,14 @@ The RetryableWriteError label might be added to an error in a variety of ways:
       * - ExceededTimeLimit
         - 262
 
-  - a server response with a write concern error response containing any of the
-    previously listed codes
+  - a mongod response with any of the previously listed codes in the
+    ``writeConcernError.code`` field.
+
+  Drivers MUST NOT add a RetryableWriteError label based on the following:
+
+  - any ``writeErrors[].code`` fields in a mongod or mongos response
+
+  - the ``writeConcernError.code`` field in a mongos response
 
   The criteria for retryable errors is similar to the discussion in the SDAM
   spec's section on `Error Handling`_, but includes additional error codes. See
@@ -854,6 +861,9 @@ inconsistent with the server and potentially confusing to developers.
 Changelog
 =========
 
+:2024-01-02: Do not use ``writeConcernError.code`` in pre-4.4 mongos response to
+             determine retryability. Do not use ``writeErrors[].code`` in
+             pre-4.4 server responses to determine retryability.
 :2023-12-06: Clarify that writes are not retried within transactions.
 :2023-12-05: Add that any server information associated with retryable
              exceptions MUST reflect the originating server, even in the

diff --git a/source/retryable-writes/tests/unified/insertOne-serverErrors.json b/source/retryable-writes/tests/unified/insertOne-serverErrors.json