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

Encryption basics #1

Draft
wants to merge 73 commits into
base: main
Choose a base branch
from
Draft

Encryption basics #1

wants to merge 73 commits into from

Conversation

rok
Copy link
Owner

@rok rok commented Jan 24, 2025

Encryption preview

@rok rok force-pushed the encryption-basics branch from 7faa8bf to a52607e Compare January 24, 2025 16:32
@rok rok force-pushed the encryption-basics branch from a017bda to 9b8662e Compare February 7, 2025 15:32
@rok rok force-pushed the encryption-basics branch from 9b8662e to 16ec681 Compare February 10, 2025 12:22
rok and others added 10 commits February 10, 2025 13:24
Use ParquetMetaDataReader

Fix CI

test

save progress

work

Review feedback

page decompression issue

add update_aad

Change encrypt and decrypt to return Results

Use correct page ordinal and module type in AADs

Tidy up ordinal types

Lint

Fix regular deserialization path

cleaning

Update data checks in test

start non-uniform decryption

Add missing doc comments

Make encryption an optional feature

Handle when a file is encrypted but encryption is disabled or no decryption properties are provided

Allow for plaintext footer

work

Fix method name

work

Minor

work

work

work

Fix reading to end of file

Refactor tests

Fix non-uniform encryption configuration

Don't use footer key for non-encrypted columns

Rebase and cleanup

Cleanup

Cleanup

Cleanup

Cleanup

Cleanup

Cleanup

lint

Remove encryption setup

Fix building with ring on wasm

file_decryptor into a seperate module

lint

FileDecryptionProperties should have at least one key

Move cyphertext reading into decryptor

More tidy up of footer key handling

Get column decryptors as RingGcmBlockDecryptor

Use Arc<dyn BlockDecryptor>

Fix file metadata tests

Handle reading plaintext footer files without decryption properties

Split up encryption modules further

Error instead of panic for AES-GCM-CTR
Make encryption externally visible
@rok rok force-pushed the encryption-basics branch from 16ec681 to 8f94e2c Compare February 10, 2025 12:25
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.

4 participants