mlswg / mls-protocol

MLS protocol
https://messaginglayersecurity.rocks
Other
233 stars 60 forks source link

Remove protocol version from KeyPackage. #876

Closed Bren2010 closed 1 year ago

Bren2010 commented 1 year ago

The protocol version is already explicitly provided by the enclosing MLSMessage, and signed by SignWithLabel.

bifurcation commented 1 year ago

Closing this after discussion with @raphaelrobert. While in principle, a few bytes could be saved, it's a very minor improvement, and not enough to justify changing a signed structure this late in the process.