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 At Rest #284

Merged
merged 6 commits into from
Nov 7, 2023
Merged

Encryption At Rest #284

merged 6 commits into from
Nov 7, 2023

Conversation

AnalogJ
Copy link
Member

@AnalogJ AnalogJ commented Oct 18, 2023

Migrated from github.com/glebarez/sqlite to gorm.io/driver/sqlite (which uses github.com/mattn/go-sqlite3)

Override github.com/mattn/go-sqlite3 with forked version from @jgiannuzzi which supports Encryption at rest. See mattn/go-sqlite3#1109
See #236

Added documentation for how to open an encrypted sqlite database in IntelliJ - CONTRIBUTING.md update


fixes #236

@AnalogJ
Copy link
Member Author

AnalogJ commented Oct 18, 2023

I made a small change to gorm_common.go that you may want to keep an eye on @nickmurray47

@AnalogJ AnalogJ mentioned this pull request Oct 18, 2023
@AnalogJ AnalogJ changed the title moving sqlitePragmaString function to SqliteRepository file. Encryption At Rest Oct 27, 2023
Migrated from github.com/glebarez/sqlite to gorm.io/driver/sqlite (which uses github.com/mattn/go-sqlite3)

Override github.com/mattn/go-sqlite3 with forked version from @jgiannuzzi which supports Encryption at rest.
See mattn/go-sqlite3#1109
See #236

Added documentation for how to open an encrypted sqlite database in IntelliJ - CONTRIBUTING.md update
- verify invalid/short database.encryption.key values throw an error
- make sure that we mock database.encryption.key checks in all tests.
- make sure we always delete the -shm and -wal temp files.
TODO: TestListBackgroundJobs and TestUpdateBackgroundJob broken
@AnalogJ AnalogJ merged commit 77cb814 into main Nov 7, 2023
7 checks passed
@AnalogJ AnalogJ deleted the encryption_at_rest branch November 7, 2023 23:17
@AnalogJ AnalogJ mentioned this pull request Nov 7, 2023
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.

Encryption at rest
1 participant