Skip to content

V2 Integration Test Issues #3246

Open
Open
@goatgoose

Description

@goatgoose

Problem:

The following issues were discovered when trying to run V2 tests with additional libcryptos in #3244. Any tests that were affected by these problems were temporarily disabled. When these issues are resolved, however, the tests that were disabled should be restored.

Solution:

For each issue, ensure that the behavior is not expected. If it isn't, fix the problem, and then enable testing support for the issue in V2.

  • Does this change what S2N sends over the wire? If yes, explain.
  • Does this change any public APIs? If yes, explain.
  • Which versions of TLS will this impact?

Requirements / Acceptance Criteria:

All issues are addressed, either by finding issues to demonstrate expected behavior, or by resolving the problem in the issue and adding test support for the issue in V2.

  • RFC links: Links to relevant RFC(s)
  • Related Issues: Link any relevant issues
  • Will the Usage Guide or other documentation need to be updated?
  • Testing: How will this change be tested? Call out new integration tests, functional tests, or particularly interesting/important unit tests.
    • Will this change trigger SAW changes? Changes to the state machine, the s2n_handshake_io code that controls state transitions, the DRBG, or the corking/uncorking logic could trigger SAW failures.
    • Should this change be fuzz tested? Will it handle untrusted input? Create a separate issue to track the fuzzing work.

Out of scope:

N/A

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions