Uploaded image for project: 'C# Driver'
  1. C# Driver
  2. CSHARP-3742

Test serialization of BSON with embedded null bytes in strings

XMLWordPrintableJSON

    • Type: Icon: Spec Change Spec Change
    • Resolution: Fixed
    • Priority: Icon: Major - P3 Major - P3
    • 2.25.0
    • Affects Version/s: None
    • Component/s: None
    • Fully Compatible
    • Not Needed
    • Hide

      1. What would you like to communicate to the user about this feature?
      2. Would you like the user to see examples of the syntax and/or executable code and its output?
      3. Which versions of the driver/connector does this apply to?

      Show
      1. What would you like to communicate to the user about this feature? 2. Would you like the user to see examples of the syntax and/or executable code and its output? 3. Which versions of the driver/connector does this apply to?

      Downstream changes: drivers should sync BSON corpus spec tests with mongodb/specifications@ff5d91f. Specifically:

      • There are new decodeErrors tests for document.json, regex.json, and top.json
      • There are new parseErrors tests for top.json
      • There are new prose tests for encoding BSON documents and regex types.

      Existing language in the spec was also relaxed to no longer require drivers to successfully parse parseErrors tests in binary.json and top.json with a non-Extended JSON parser (to check for general JSON syntax errors). Also, an allowance was made for drivers that parse Extended JSON directly into language types (with regard to evaluating parseErrors).

      Per DRIVERS-101, drivers are expected to validate that C strings do not contain embedded null bytes.

      We should add spec or prose tests to verify BSON C strings with embedded null bytes are rejected when serialized.

            Assignee:
            james.kovacs@mongodb.com James Kovacs
            Reporter:
            backlog-server-pm Backlog - Core Eng Program Management Team
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: