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

Signed ECIES Payloads #100

Merged
merged 6 commits into from
Oct 4, 2023
Merged

Signed ECIES Payloads #100

merged 6 commits into from
Oct 4, 2023

Conversation

neekolas
Copy link
Collaborator

@neekolas neekolas commented Oct 4, 2023

Summary

  • Adds a new SignedPayload message type to hold arbitrary bytes and a signature
  • Adds a wrapper type for EciesMessage, which currently just holds a v1 structure for encrypted SignedPayload messages

@neekolas neekolas marked this pull request as ready for review October 4, 2023 19:05
@neekolas neekolas requested a review from a team as a code owner October 4, 2023 19:05
@neekolas neekolas changed the base branch from main to xmtpv3 October 4, 2023 19:08
@neekolas neekolas merged commit c03de5c into xmtpv3 Oct 4, 2023
8 checks passed
@neekolas neekolas deleted the nmolnar/signed-payload branch October 4, 2023 19:27
@richardhuaaa
Copy link
Contributor

@neekolas this is for the v2 portable preferences right? Does this belong in main?

@neekolas
Copy link
Collaborator Author

neekolas commented Oct 4, 2023

You would think so, but libxmtp only looks at the v3 branch. Including in libxmtp's V2 branch.

We could change that, since it's quite confusing.

@richardhuaaa
Copy link
Contributor

Ahhh gotcha. Fine with leaving it as-is for now

@neekolas
Copy link
Collaborator Author

neekolas commented Oct 4, 2023

Really what I'd like to do is kill the v3 proto branch. None of our v3 work to date has involved changing any v2 protos, and I don't think it will. So the separate branching strategy isn't protecting us from much.

@github-actions
Copy link

🎉 This PR is included in version 3.28.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants