Skip to content

feat: Improve API for Encryption with a session key#331

Merged
lubux merged 1 commit intomainfrom
feat/crypto-refresh-session-key-handling
Apr 10, 2025
Merged

feat: Improve API for Encryption with a session key#331
lubux merged 1 commit intomainfrom
feat/crypto-refresh-session-key-handling

Conversation

@lubux
Copy link
Member

@lubux lubux commented Apr 9, 2025

With RFC 9580, clients can encrypt data using AEAD with SEIPDv2 packets. However, SEIPDv2 packets are only compatible with PKESK v6 and SKESK v6 packets when the session key is encrypted using an OpenPGP certificate.

This PR improves the API to reduce the risk of encrypting data with SEIPDv2 while using a session key encrypted in a packet version below v6. Specifically, the session key now includes an indicator of whether it is intended for AEAD use or not and affects the produced packets.

@lubux lubux force-pushed the feat/crypto-refresh-session-key-handling branch 2 times, most recently from f344d7b to 1b2b9fa Compare April 10, 2025 07:12
@lubux lubux force-pushed the feat/crypto-refresh-session-key-handling branch from 1b2b9fa to 63f920c Compare April 10, 2025 14:10
With RFC 9580, clients can encrypt data using AEAD with SEIPDv2 packets.
However, SEIPDv2 packets are only compatible with PKESK v6 and
SKESK v6 packets when the session key is encrypted using an OpenPGP certificate.

This commit improves the API to reduce the risk of encrypting data with SEIPDv2
while using a session key encrypted in a packet version below v6.
Specifically, the session key now includes an indicator of whether it is intended for AEAD use or not
and affects the produced packets.
@lubux lubux force-pushed the feat/crypto-refresh-session-key-handling branch from 63f920c to f0279bb Compare April 10, 2025 14:11
@lubux lubux merged commit 2d743b4 into main Apr 10, 2025
9 checks passed
@lubux lubux deleted the feat/crypto-refresh-session-key-handling branch April 10, 2025 14:18
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.

2 participants