Skip to content

[PLACEHOLDER] wire/stable version - #81

Draft
OtaK wants to merge 56 commits into
feat/rfc9420from
wire/stable
Draft

[PLACEHOLDER] wire/stable version#81
OtaK wants to merge 56 commits into
feat/rfc9420from
wire/stable

Conversation

@OtaK

@OtaK OtaK commented Feb 14, 2024

Copy link
Copy Markdown

No description provided.

@github-actions github-actions Bot added size/xl and removed size/l labels Apr 15, 2024
@github-actions

Copy link
Copy Markdown

This PR is so big! Please, split it 😊

@github-actions

Copy link
Copy Markdown

This PR is so big! Please, split it 😊

Kyber PQ KEM is still being worked on and has an active draft:
https://datatracker.ietf.org/doc/html/draft-cfrg-schwabe-kyber-04.

Additionally, the hybrid scheme that combines X25519 and Kyber is
also in the drafting phase:
https://www.ietf.org/archive/id/draft-tls-westerbaan-xyber768d00-02.html

Given this, it makes no sense for us to provide support for Kyber-based
KEMs.
@github-actions

github-actions Bot commented Jul 4, 2024

Copy link
Copy Markdown

This PR is so big! Please, split it 😊

@github-actions

Copy link
Copy Markdown

This PR is so big! Please, split it 😊

They're just too big and overflow the stack when they run.
@github-actions

Copy link
Copy Markdown

This PR is so big! Please, split it 😊

@github-actions

Copy link
Copy Markdown

This PR is so big! Please, split it 😊

@github-actions

Copy link
Copy Markdown

This PR is so big! Please, split it 😊

This allows creating a commit from proposals without sending the
proposals over the wire first.
Previously, the tree shrink function would be called eagerly after each
remove proposal. If the shrink threshold was reached, the blanked out
nodes were removed from the diff tree.

If there were additional add proposals in a commit that triggered the
tree to grow again, this resulted in incorrect state if there were FEWER
add proposals than remove proposals. That's because the blanked out nodes
removed during shrinking were not re-added during growing.

To fix this, we don't shrink/trim the tree eagerly after each remove
proposal, but after remove and add proposals have been processed.
This proves that we don't suffer from the bug fixed in
openmls/openmls@26eaa4d0e
@CLAassistant

CLAassistant commented Jun 11, 2026

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 4 committers have signed the CLA.

❌ beltram
❌ coriolinus
❌ OtaK
❌ franziskuskiefer
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

8 participants