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

[Hackathon117] Add encrypted personalization data using ES-ECDH #351

Merged
merged 6 commits into from Nov 4, 2023

Conversation

kentakayama
Copy link
Contributor

Solves #330
NOTE: I've generated this binary using t_cose 2 alpha but its status is still fixing some bugs.
Please wait until I can validate this.

@kentakayama kentakayama changed the title WIP: Add encrypted personalization data using ES-ECDH [Hackathon117] WIP: Add encrypted personalization data using ES-ECDH Jul 24, 2023
@kentakayama
Copy link
Contributor Author

  • ✔️ This example is validated by processing with the latest libcsuit.
  • ⚠️ Need to test with other implementation to decrypt the COSE_Encrypt.

@hannestschofenig
Copy link
Collaborator

I think an example would be very useful!

@kentakayama
Copy link
Contributor Author

This example has updated to use KDF context and now ready to merge.

@kentakayama kentakayama marked this pull request as ready for review September 10, 2023 23:39
@kentakayama kentakayama changed the title [Hackathon117] WIP: Add encrypted personalization data using ES-ECDH [Hackathon117] Add encrypted personalization data using ES-ECDH Sep 11, 2023
Copy link
Collaborator

@hannestschofenig hannestschofenig left a comment

Choose a reason for hiding this comment

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

The PR looks good to me.

@dthaler dthaler mentioned this pull request Oct 6, 2023
@mingpeiwk
Copy link
Collaborator

Looks good for me too.

@hannestschofenig
Copy link
Collaborator

Can we merge this PR?

@dthaler
Copy link
Collaborator

dthaler commented Oct 19, 2023

In my view this should not be merged until suit-mti changes.

@dthaler dthaler merged commit 655241b into ietf-teep:master Nov 4, 2023
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

4 participants