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

feat: private keys support for testing #159

Merged
merged 18 commits into from
Dec 3, 2024

Conversation

segfault-magnet
Copy link
Contributor

@segfault-magnet segfault-magnet commented Nov 19, 2024

closes: #160

We'll still be using KMS in our e2e tests so that we remain as close to prod as possible.

The bridge repo can now specify their keys as "Private(key_here)" while we'll configure the keys as "Kms(arn_here)".

@segfault-magnet segfault-magnet self-assigned this Nov 19, 2024
@segfault-magnet segfault-magnet marked this pull request as ready for review November 19, 2024 14:40
hal3e
hal3e previously approved these changes Nov 26, 2024
Copy link
Contributor

@hal3e hal3e left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work. Left some suggestions

e2e/src/whole_stack.rs Outdated Show resolved Hide resolved
packages/eth/src/websocket.rs Outdated Show resolved Hide resolved
MujkicA
MujkicA previously approved these changes Nov 27, 2024
packages/eth/src/websocket.rs Show resolved Hide resolved
README.md Show resolved Hide resolved
@segfault-magnet segfault-magnet merged commit 500f8ad into master Dec 3, 2024
13 checks passed
@segfault-magnet segfault-magnet deleted the feat/private_keys_support_for_testing branch December 3, 2024 12:06
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.

make the committer accept private keys for testing
3 participants