Skip to content
This repository has been archived by the owner on Jul 2, 2023. It is now read-only.

Feature: add openssl and ring to implement block cipher #62

Merged
merged 2 commits into from
Apr 22, 2023

Conversation

Xynnn007
Copy link
Member

@Xynnn007 Xynnn007 commented Apr 7, 2023

ring crate can provide the same random generator function
as openssl. We add two features to implement rand_bytes

  • block-cipher-ring: based on ring, which is purely rust but
    cannot be built on some arch like s390x.
  • block-cipher-openssl: based on vendored openssl, but it
    requires Make, C compiler and perl.

Fixes: confidential-containers/guest-components#255

`ring` crate can provide the same random generator function
as `openssl`. We add two features to implement rand_bytes

- `block-cipher-ring`: based on ring, which is purely rust but
cannot be built on some arch like `s390x`.
- `block-cipher-openssl`: based on vendored openssl, but it
requires `Make`, `C` compiler and `perl`.

Fixes: #61

Signed-off-by: Xynnn007 <xynnn@linux.alibaba.com>
now either openssl or ring is used to implement block
cipher module.

Signed-off-by: Xynnn007 <xynnn@linux.alibaba.com>
@Xynnn007 Xynnn007 changed the title dep: get rid of openssl Feature: add openssl and ring to implement block cipher Apr 7, 2023
@Xynnn007 Xynnn007 marked this pull request as ready for review April 17, 2023 06:36
@Xynnn007
Copy link
Member Author

Could you help to review this PR? @jiangliu @arronwy

Copy link
Member

@arronwy arronwy left a comment

Choose a reason for hiding this comment

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

Thanks @Xynnn007 LGTM!

@arronwy arronwy merged commit 076c7bd into confidential-containers:main Apr 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Features to distinguish openssl and purely-rust(including ring)
2 participants