Skip to content

[Crypto][BSSL] Add boringssl based implementation of the crypto interface#32

Merged
werwurm merged 63 commits intomainfrom
werwurm/bssl_crypto_impl
Jan 23, 2025
Merged

[Crypto][BSSL] Add boringssl based implementation of the crypto interface#32
werwurm merged 63 commits intomainfrom
werwurm/bssl_crypto_impl

Conversation

@werwurm
Copy link
Contributor

@werwurm werwurm commented Dec 20, 2024

This PR adds a reference implementation of the nat20 crypto interface based on boringssl. It also instantiates the crypto test and runs it against the implementation.

@werwurm werwurm requested a review from a team as a code owner January 8, 2025 21:51
timhirsh
timhirsh previously approved these changes Jan 9, 2025
Copy link
Member

@timhirsh timhirsh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.github/workflows/cmake-build-and-test.yml LGTM 👍 Deferring to other subject matter experts for the remaining changes.

@werwurm werwurm changed the base branch from werwurm/crypto_test to main January 14, 2025 18:58
@werwurm werwurm dismissed timhirsh’s stale review January 14, 2025 18:58

The base branch was changed.

@werwurm werwurm requested a review from seidelrj January 16, 2025 21:30
Copy link
Contributor

@seidelrj seidelrj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just 1 copyright to fix.

@werwurm werwurm requested a review from seidelrj January 23, 2025 18:54
@werwurm werwurm requested a review from timhirsh January 23, 2025 22:05
@werwurm werwurm merged commit 3880a54 into main Jan 23, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants