Skip to content

Add E2E tests with OpenSSL #11

Closed
Closed

Description

@Sean-Der

Add some basic testing using s_client + s_server

The author should also refactor/re-use our existing E2E testing code so that we don't duplicate in both places.

Activity

  1. Sean-Der commented on Dec 3, 2018

    @Sean-Der
    MemberAuthor

    Also introduce packet loss on either side. Create a Client/Server from a wrapped Conn and add a certain % of packet loss (it may be helpful to log messages, otherwise fixing will be really hard here)

  2. backkem commented on Dec 3, 2018

    @backkem
    Member

    We should probably implement these types of tests globally so we can throw them at any layer of the stack. The beginnings of this are here. Eventually it could look like this.

  3. added a commit that references this issue on Dec 10, 2018
  4. changed the title [-]Implement integration testing[/-] [+]Implement E2E testing[/+] on Dec 11, 2018
  5. changed the title [-]Implement E2E testing[/-] [+]Add E2E tests with OpenSSL[/+] on Dec 11, 2018
  6. 5 remaining items

  7. igolaizola commented on Feb 24, 2020

    @igolaizola
    Member

    OpenSSL tests have been added in #193

    TODO:

    • Make OpenSSL work with ed25519 test
  8. at-wat commented on Mar 11, 2020

    @at-wat
    Member

    I found that our implementation of Ed25519 signature calculation and hash used in ServerKeyExchange/CertificateVerify messages are wrong. Since both of our server and client have same bug, they can communicate each other.
    I have made a fix for it, and now I'm cleaning the code.

    One bad news is that OpenSSL haven't support DTLS Ed25519 yet.
    I could confirm my fix works with this openssl branch: openssl/openssl#11115

  9. at-wat commented on Mar 12, 2020

    @at-wat
    Member

    Ed25519 implementation of the master branch should be fixed now. We can enable the E2E test once OpenSSL supports DTLS 1.2 Ed25519.

  10. added
    waiting:upstreamWaiting on a fix in an upstream project/depedency
    and removed
    triagedHas been reviewed
    on Mar 12, 2020
  11. added 5 commits that reference this issue on May 17, 2022
    a83651c
    062c3b6
    840e6db
    e8c6a4c
    435885c
  12. removed
    waiting:upstreamWaiting on a fix in an upstream project/depedency
    on May 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      ⌂ 换网址