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

feat: add worktop/crypto module #11

Merged
merged 4 commits into from
Mar 23, 2021
Merged

feat: add worktop/crypto module #11

merged 4 commits into from
Mar 23, 2021

Conversation

lukeed
Copy link
Owner

@lukeed lukeed commented Mar 22, 2021

This is a draft PR while I'm collecting feedback (and still extracting) for which other utility functions should be included here.

So far, this contains:

  • digest(algo, message)
  • SHA1(message)
  • SHA256(message)
  • SHA384(message)
  • SHA512(message)

Planned/Maybe'd:

  • importkey(secret, algo, scopes?)
  • verify(secret, algo, message)
  • sign(secret, algo, message)

BIG Maybe

NOTE: I have this, but it might be too specific to me?

  • PBKDF2(input, salt, iterations, keylen, digest)

What else should be here? 🙏

@saibotsivad
Copy link

The ones you have so far sound great, and I know I personally would use PBKDF2 on several projects, so that would be a big help for me 🤷

I don't think I use key stuff (the maybes) very often if at all, in most of the business apps I build.

@lukeed
Copy link
Owner Author

lukeed commented Mar 22, 2021

I think I'll merge what's here & move the Maybe items to an issue for continued brainstorming.

@lukeed lukeed marked this pull request as ready for review March 23, 2021 01:46
@lukeed lukeed merged commit d8ee779 into master Mar 23, 2021
@lukeed lukeed deleted the feat/crypto branch March 23, 2021 01:54
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.

2 participants