-
Notifications
You must be signed in to change notification settings - Fork 10
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
feat: AWS KMS for delegation #1435
Merged
Merged
Changes from 38 commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
44ff4ef
feat: first commit
freemanzMrojo 706c68e
feat: second commit
freemanzMrojo d45caaf
feat: second commit
freemanzMrojo 6940798
feat: second commit
freemanzMrojo 64b375f
feat: second commit
freemanzMrojo fbf7a6b
feat: second commit
freemanzMrojo b2a10d9
feat: more changes
freemanzMrojo 987f0e4
feat: more changes
freemanzMrojo 9c1864b
feat: more changes
freemanzMrojo e981f81
feat: more changes
freemanzMrojo e445381
feat: more changes
freemanzMrojo c0dd185
feat: more changes
freemanzMrojo e6fe929
feat: more changes
freemanzMrojo 465a9c6
feat: more changes
freemanzMrojo 929df4e
feat: scaffolding for testing
freemanzMrojo fd29372
feat: scaffolding for testing
freemanzMrojo efa6946
feat: scaffolding for testing
freemanzMrojo 2393dd6
feat: added gitleaks exceptions
freemanzMrojo 6cfeda5
Merge branch 'main' of https://github.com/vechain/vechain-sdk-js into…
freemanzMrojo 129554f
feat: some more changes
freemanzMrojo e033966
feat: some more changes
freemanzMrojo c727a54
feat: solo tests not working
freemanzMrojo 0334d16
feat: first tests working
freemanzMrojo 4a2d74a
feat: more changes
freemanzMrojo 8dcce93
feat: still delegation url to test
freemanzMrojo ad36cf2
feat: still delegation url to test
freemanzMrojo e348ea5
feat: still delegation url to test
freemanzMrojo 0153438
feat: still delegation url to test
freemanzMrojo 969cf55
feat: still delegation url to test
freemanzMrojo 0b3e99b
feat: fixed delegated hash
freemanzMrojo 4f5b430
feat: more tests
freemanzMrojo 7ff51f6
feat: added testnet tests
freemanzMrojo 10b98dd
Merge branch 'main' of https://github.com/vechain/vechain-sdk-js into…
freemanzMrojo cddbed5
feat: modified readme
freemanzMrojo 6d06668
feat: modified readme
freemanzMrojo f6fde33
feat: modified readme
freemanzMrojo d8d82f9
Merge branch 'main' into feat/1413-aws-kms-delegator
freemanzMrojo 1ef2d71
feat: modified readme
freemanzMrojo a35024c
Merge branch 'main' into feat/1413-aws-kms-delegator
fabiorigam 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1,2 @@ | ||
# Exceptions for gitleaks to be placed here | ||
465a9c6b265ec49015fd7ff641506905cb93811c:packages/aws-kms-adapter/tests/test-aws-credentials.json:generic-api-key:3 | ||
465a9c6b265ec49015fd7ff641506905cb93811c:packages/aws-kms-adapter/tests/test-aws-credentials.json:generic-api-key:12 |
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,8 +1,14 @@ | ||
#!/bin/bash | ||
|
||
CUSTOM_KEY_MATERIAL="f5KQzETF/SuV/iHWrW/l+pwXfhzW87TJapexPgnqoVg=" | ||
CUSTOM_ID="bffb20d8-35ca-4408-9d54-f775b929b38d" | ||
CUSTOM_ORIGIN_KEY_MATERIAL="f5KQzETF/SuV/iHWrW/l+pwXfhzW87TJapexPgnqoVg=" | ||
CUSTOM_ORIGIN_ID="bffb20d8-35ca-4408-9d54-f775b929b38d" | ||
CUSTOM_DELEGATOR_ID="3e47cac8-de37-4f50-b591-57f525c1b05c" | ||
|
||
# The command succeeds but the key is not created with the custom key material (should be fixed once this is clarified: https://github.com/localstack/localstack/issues/11678) | ||
# awslocal kms create-key --key-usage SIGN_VERIFY --key-spec ECC_SECG_P256K1 --tags "[{\"TagKey\":\"_custom_key_material_\",\"TagValue\":\"$CUSTOM_KEY_MATERIAL\"},{\"TagKey\":\"_custom_id_\",\"TagValue\":\"$CUSTOM_ID\"}]" | ||
awslocal kms create-key --key-usage SIGN_VERIFY --key-spec ECC_SECG_P256K1 --tags "[{\"TagKey\":\"_custom_id_\",\"TagValue\":\"$CUSTOM_ID\"}]" | ||
# awslocal kms create-key --key-usage SIGN_VERIFY --key-spec ECC_SECG_P256K1 --tags "[{\"TagKey\":\"_custom_key_material_\",\"TagValue\":\"$CUSTOM_ORIGIN_KEY_MATERIAL\"},{\"TagKey\":\"_custom_id_\",\"TagValue\":\"$CUSTOM_ORIGIN_ID\"}]" | ||
|
||
# Origin key | ||
awslocal kms create-key --key-usage SIGN_VERIFY --key-spec ECC_SECG_P256K1 --tags "[{\"TagKey\":\"_custom_id_\",\"TagValue\":\"$CUSTOM_ORIGIN_ID\"}]" | ||
|
||
# Delegator key | ||
awslocal kms create-key --key-usage SIGN_VERIFY --key-spec ECC_SECG_P256K1 --tags "[{\"TagKey\":\"_custom_id_\",\"TagValue\":\"$CUSTOM_DELEGATOR_ID\"}]" |
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.
Why is this command commented?
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.
It is because of this issue localstack/localstack#11678 . Basically nowadays we cannot import a custom key into localstack, but once the issue is fixed we should be able to do it with this commented-out line.
The command itself does not break but it does not import anything.