Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Handling some minor comments on the SECP changes #313

Merged
merged 5 commits into from
Oct 5, 2022
Merged

Handling some minor comments on the SECP changes #313

merged 5 commits into from
Oct 5, 2022

Commits on Oct 4, 2022

  1. Add deserialization tests, pin repro

    @kozross @iquerejeta
    kozross authored and iquerejeta committed Oct 4, 2022
    Configuration menu
    Copy the full SHA
    709b9be View commit details
    Browse the repository at this point in the history

  2. Add length check to Ed25519 deserialization

    @kozross @iquerejeta
    kozross authored and iquerejeta committed Oct 4, 2022
    Configuration menu
    Copy the full SHA
    71cf640 View commit details
    Browse the repository at this point in the history

  3. Handling audit comments

    @iquerejeta
    iquerejeta committed Oct 4, 2022
    Configuration menu
    Copy the full SHA
    72cca44 View commit details
    Browse the repository at this point in the history

Commits on Oct 5, 2022

  1. Apply suggestions from code review 

    Co-authored-by: Alexey Kuleshevich <alexey.kuleshevich@iohk.io>
    @iquerejeta @lehins
    iquerejeta and lehins committed Oct 5, 2022
    Configuration menu
    Copy the full SHA
    858c33e View commit details
    Browse the repository at this point in the history

  2. Remove trailing whitespaces

    @iquerejeta
    iquerejeta committed Oct 5, 2022
    Configuration menu
    Copy the full SHA
    5836efa View commit details
    Browse the repository at this point in the history