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

Add basic HMAC and HKDF functions #38

Merged
merged 15 commits into from
Sep 19, 2022
Merged

Add basic HMAC and HKDF functions #38

merged 15 commits into from
Sep 19, 2022

Conversation

benr-ml
Copy link
Contributor

@benr-ml benr-ml commented Sep 12, 2022

Both use SHA256 as the underlying hash function.

#39

@benr-ml benr-ml linked an issue Sep 13, 2022 that may be closed by this pull request
@benr-ml benr-ml removed a link to an issue Sep 13, 2022
@benr-ml benr-ml marked this pull request as ready for review September 13, 2022 08:16
@joyqvq
Copy link
Collaborator

joyqvq commented Sep 13, 2022

lgtm! thanks for the comprehensive test cases, we can perhaps port over https://github.com/google/wycheproof/blob/master/testvectors/hmac_sha256_test.json wycheproof test cases in the future

joyqvq
joyqvq previously approved these changes Sep 13, 2022
src/hmac.rs Outdated Show resolved Hide resolved
src/hmac.rs Show resolved Hide resolved
src/hmac.rs Show resolved Hide resolved
src/hmac.rs Outdated Show resolved Hide resolved
src/hmac.rs Outdated Show resolved Hide resolved
kchalkias
kchalkias previously approved these changes Sep 15, 2022
src/hmac.rs Outdated Show resolved Hide resolved
Copy link
Collaborator

@kchalkias kchalkias left a comment

Choose a reason for hiding this comment

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

@benr-ml let's please rebase before merging; after recent merges there are now small conflicts.

Copy link
Collaborator

@kchalkias kchalkias left a comment

Choose a reason for hiding this comment

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

Awesome job!

@benr-ml benr-ml merged commit 8cec139 into main Sep 19, 2022
@benr-ml benr-ml deleted the hmac_hkdf branch September 19, 2022 16:49
jonas-lj added a commit that referenced this pull request Sep 26, 2022
commit 6c679f7
Author: Jonas Lindstrøm <jonas-lj@users.noreply.github.com>
Date:   Fri Sep 23 20:43:55 2022 +0200

    Include hash functions (SHA-2, SHA-3, Keccak and SHAKE2) (#63)

commit 4b7a06f
Merge: 49b6cf7 aa215c8
Author: Joy Wang <108701016+joyqvq@users.noreply.github.com>
Date:   Mon Sep 19 16:30:18 2022 -0400

    Merge pull request #52 from MystenLabs/ci1

    Fix publish.yml

commit aa215c8
Author: Joy Wang <108701016+joyqvq@users.noreply.github.com>
Date:   Mon Sep 19 16:17:53 2022 -0400

    remove public-api from workflow

commit 5550248
Author: Joy Wang <108701016+joyqvq@users.noreply.github.com>
Date:   Sun Sep 18 20:03:41 2022 -0400

    ci fix

commit 49b6cf7
Author: benr-ml <112846738+benr-ml@users.noreply.github.com>
Date:   Mon Sep 19 21:09:14 2022 +0300

    update public-api (#59)

commit 8cec139
Author: benr-ml <112846738+benr-ml@users.noreply.github.com>
Date:   Mon Sep 19 19:49:47 2022 +0300

    Add basic HMAC and HKDF functions (#38)
jonas-lj added a commit that referenced this pull request Sep 26, 2022
* Bump blake2 from 0.9.2 to 0.10.4

Bumps [blake2](https://github.com/RustCrypto/hashes) from 0.9.2 to 0.10.4.
- [Release notes](https://github.com/RustCrypto/hashes/releases)
- [Commits](RustCrypto/hashes@blake2-v0.9.2...blake2-v0.10.4)

---
updated-dependencies:
- dependency-name: blake2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* Squashed commit of the following:

commit 6c679f7
Author: Jonas Lindstrøm <jonas-lj@users.noreply.github.com>
Date:   Fri Sep 23 20:43:55 2022 +0200

    Include hash functions (SHA-2, SHA-3, Keccak and SHAKE2) (#63)

commit 4b7a06f
Merge: 49b6cf7 aa215c8
Author: Joy Wang <108701016+joyqvq@users.noreply.github.com>
Date:   Mon Sep 19 16:30:18 2022 -0400

    Merge pull request #52 from MystenLabs/ci1

    Fix publish.yml

commit aa215c8
Author: Joy Wang <108701016+joyqvq@users.noreply.github.com>
Date:   Mon Sep 19 16:17:53 2022 -0400

    remove public-api from workflow

commit 5550248
Author: Joy Wang <108701016+joyqvq@users.noreply.github.com>
Date:   Sun Sep 18 20:03:41 2022 -0400

    ci fix

commit 49b6cf7
Author: benr-ml <112846738+benr-ml@users.noreply.github.com>
Date:   Mon Sep 19 21:09:14 2022 +0300

    update public-api (#59)

commit 8cec139
Author: benr-ml <112846738+benr-ml@users.noreply.github.com>
Date:   Mon Sep 19 19:49:47 2022 +0300

    Add basic HMAC and HKDF functions (#38)

* Update usage of blake2b crate to new version

* Fix fmt

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jonas Lindstrøm <jonas-lj@users.noreply.github.com>
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