forked from postgres/postgres
-
Notifications
You must be signed in to change notification settings - Fork 7
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
PG-1457 Key management funcs renaming #126
Open
artemgavrilov
wants to merge
17
commits into
release-17.4
Choose a base branch
from
PG-1457-key-management-funcs-renaming
base: release-17.4
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
17 commits
Select commit
Hold shift + click to select a range
410a727
PG-1457 Rename some key management funcions
artemgavrilov 6d5a5c9
PG-1457 Fix some tests
artemgavrilov 74abe00
PG-1457 Hit CI
artemgavrilov 39119bf
PG-1457 Rename key in CI setup
artemgavrilov 481df25
PG-1457 Rename pg_tde_verify_global_principal_key to pg_tde_verify_se…
artemgavrilov eb70b72
PG-1457 Rename keys in tests
artemgavrilov 66db9c3
PG-1457 Renaming
artemgavrilov e828ec9
PG-1457 Renaming
artemgavrilov dea753a
PG-1457 Fix tests
artemgavrilov e1e05c8
PG-1457 Fix tests
artemgavrilov de945ea
PG-1457 Fix tabs
artemgavrilov 59dc51d
PG-1457 Fix tests
artemgavrilov 8420e2e
PG-1457 Fix tests
artemgavrilov ec271ee
PG-1457 Fix
artemgavrilov dae1443
PG-1457 Fix test
artemgavrilov 399af94
PG-1457 Fix test
artemgavrilov 3b48f45
PG-1457 Fix test
artemgavrilov File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
CREATE SCHEMA IF NOT EXISTS tde; | ||
CREATE EXTENSION IF NOT EXISTS pg_tde SCHEMA tde; | ||
SELECT pg_tde_add_key_provider_file('reg_file-vault', '/tmp/pg_tde_test_keyring.per'); | ||
SELECT pg_tde_set_principal_key('test-db-principal-key', 'reg_file-vault'); | ||
SELECT pg_tde_add_database_key_provider_file('reg_file-vault', '/tmp/pg_tde_test_keyring.per'); | ||
SELECT pg_tde_set_principal_key_using_database_key_provider('test-db-principal-key', 'reg_file-vault'); |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the server key? Maybe rewrite to what this key is used for? For WAL encryption, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For now only for WAL encryption, but in future it may be used for something else. I guess for system tables encryption. In general its purpose to encrypt entities that has server scope availability.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok thanks, I believe it's time to explain existing scopes and their usage. In a separate PR most likely
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Absolutely agree