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.

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

RFC: no_std #63

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

RFC: no_std #63

wants to merge 1 commit into from

Conversation

aep
Copy link
Contributor

@aep aep commented Aug 31, 2019

#62

the only thing that actually requires std is Cursor in src/resolvers/default.rs CipherChaChaPoly

and getrandom, which is an upstream issue in rand

both don't seem a big problem to me, since embedded users need a different resolver anyway now that hacl is gone

@aep aep force-pushed the no_std branch 2 times, most recently from c10d34c to b1c4512 Compare August 31, 2019 09:37
@mcginty
Copy link
Owner

mcginty commented Sep 1, 2019

Great! Makes sense to do this change now. I'll check it out more once it passes CI.

I'm down for re-adding HACL*, I removed it because we had an issue right before the 0.6 release and I wanted to be safe and re-integrate it after further inspection.

@dzmitry-huba
Copy link

Hello, Jake and Arvid

I was wondering if you have plans to integrate this change and enable no_std support. I would love to use your library in our project that runs inside of the TEEs.

Please let me know what you think.

Cheers!

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.

None yet

3 participants