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

DOV-7024: Improve dcrypt design documentation #1132

Merged
merged 4 commits into from
Jan 16, 2025

Conversation

fleischie
Copy link
Contributor

  • Cleanup page,
  • amend file format 2 description and
  • add exhausting documentation for the new dcrypt-decrypt.py script. (Which depending on review might not yet be fully published.)

Closes DOV-7024.

- Consolidate external links,
- create extra sub-headers for encryption/decryption sections,
- simplify enumeration syntax to reduce error potentials, and
- add dovecotlinks for internal sections.
- Shorten "ECC" to "EC" for the key type,
- add key checksum to end of key-block,
- add message integrity tag at end of message, and
- fix alignment of byte offsets as well as descriptions.

This comment has been minimized.

@fleischie fleischie force-pushed the DOV-7024/rewrite-dcrypt-decrypt branch from 1e880ab to db9eebc Compare January 15, 2025 14:54
@fleischie fleischie force-pushed the DOV-7024/rewrite-dcrypt-decrypt branch from db9eebc to 7cd1733 Compare January 16, 2025 08:57

This comment has been minimized.

@fleischie fleischie force-pushed the DOV-7024/rewrite-dcrypt-decrypt branch 2 times, most recently from a7a49c1 to bdbd979 Compare January 16, 2025 09:07
@fleischie fleischie force-pushed the DOV-7024/rewrite-dcrypt-decrypt branch from bdbd979 to 5cf453f Compare January 16, 2025 09:18
cmouse
cmouse previously approved these changes Jan 16, 2025

This comment has been minimized.

@cmouse cmouse merged commit 16da00b into dovecot:main Jan 16, 2025
5 checks passed
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