Add unified tests for code-20 passthrough

ExportXMLWordPrintableJSON

    • Type: Spec Change
    • Resolution: Unresolved
    • Priority: Unknown
    • None
    • Component/s: Retryable Reads & Writes
    • None
    • Needed
    • Hide

      Summary of necessary driver changes

      •  

      Commits for syncing spec/prose tests
      (and/or refer to an existing language POC if needed)

      •  

      Context for other referenced/linked tickets

      •  
      Show
      Summary of necessary driver changes   Commits for syncing spec/prose tests (and/or refer to an existing language POC if needed)   Context for other referenced/linked tickets  
    • $i18n.getText("admin.common.words.hide")
      Key Status/Resolution FixVersion
      CDRIVER-6477 Blocked
      CXX-3588 Blocked
      CSHARP-6238 Blocked
      GODRIVER-4160 Blocked
      JAVA-6319 Blocked
      NODE-7865 Blocked
      PYTHON-6120 Blocked
      PHPLIB-1954 Blocked
      RUBY-3964 Blocked
      RUST-2506 Blocked
      $i18n.getText("admin.common.words.show")
      #scriptField, #scriptField *{ border: 1px solid black; } #scriptField{ border-collapse: collapse; } #scriptField td { text-align: center; /* Center-align text in table cells */ } #scriptField td.key { text-align: left; /* Left-align text in the Key column */ } #scriptField a { text-decoration: none; /* Remove underlines from links */ border: none; /* Remove border from links */ } /* Add green background color to cells with FixVersion */ #scriptField td.hasFixVersion { background-color: #00FF00; /* Green color code */ } #scriptField td.willNotDo { background-color: #FF0000; /* Red color code */ } /* Center-align the first row headers */ #scriptField th { text-align: center; } Key Status/Resolution FixVersion CDRIVER-6477 Blocked CXX-3588 Blocked CSHARP-6238 Blocked GODRIVER-4160 Blocked JAVA-6319 Blocked NODE-7865 Blocked PYTHON-6120 Blocked PHPLIB-1954 Blocked RUBY-3964 Blocked RUST-2506 Blocked

      Summary

      When retryable writes raise an exception and return error code 20, the Node Driver drops the error details and outputs an unrelated error message. More data in NODE-7827.
      This was missed for 4 years, partially because this aspect of the specifications does not have associated tests. This task is to add unified tests that would have caught this rewrite issue in the Node Driver.

      Motivation

      Who is the affected end user?

      Every user who makes retryable calls and encounters exceptions.

      How does this affect the end user?

      Before, the error message being thrown by the Driver was wrong. After, the message will match what the Server sent.

      How likely is it that this problem or use case will occur?

      Error message improvement.

      If the problem does occur, what are the consequences and how severe are they?

      Wrong message in the exception. Node Driver has had this for 4 years, and we just got notified, so this is low severity.

      Is this issue urgent?

      No. Node Driver has had this issue for 4 years, we just got notified about it.

      Is this ticket required by a downstream team?

      No.

      Is this ticket only for tests?

      Yes.

      Acceptance Criteria

      Add tests to verify code-20 passthrough.

            Assignee:
            Ferdinando Papale
            Reporter:
            Pavel Safronov
            Ferdinando Papale Ferdinando Papale
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: