-
Notifications
You must be signed in to change notification settings - Fork 246
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: signers #44
Merged
Merged
feat: signers #44
Changes from all commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
eb44578
init
DaniPopes 809a4d3
setup
DaniPopes ed1d797
wip
DaniPopes fd82f7b
cfg stuff
DaniPopes f8c6554
chore: clippy
DaniPopes 15ddd9b
signature methods, tests
DaniPopes 5ff9a66
fix mnemonic test
DaniPopes a9a3685
fixes
DaniPopes 9a90a52
wip: split to crates
DaniPopes ebccc56
fixes
DaniPopes 9b0952d
chore: clippy
DaniPopes 49e8a35
sort
DaniPopes 8254a51
chore: consolidate aws to a single file
DaniPopes 1d7aa37
rename `{Ledger,Trezor}` to `{Ledger,Trezor}Signer`
DaniPopes d5e3aa4
restructure
DaniPopes 42e1824
add `set_chain_id`, provide a default `with_chain_id`
DaniPopes 01b6086
bump AWS to 1.0
DaniPopes 7a27dfb
better default impls for trait methods, sync feature
DaniPopes e6b16b5
remove Error GAT
DaniPopes 0514193
remove chain_id
DaniPopes 2e5734d
chore: clippy
DaniPopes 19328b4
feats
DaniPopes e2709bf
msrv
DaniPopes 3878e88
deps
DaniPopes 49961f2
wazm
DaniPopes 2ec2b60
stop failing
DaniPopes a8a946b
asynctrait
DaniPopes ec9cf02
tracing
DaniPopes 26bf3b6
update ledger
DaniPopes 447982c
Revert "remove chain_id"
DaniPopes 493538b
split signer into two traits
DaniPopes 579717d
make async the default method name
DaniPopes f814a92
docs
DaniPopes d701a79
make signature copy
DaniPopes 61911f6
wallet error module
DaniPopes 05d1978
docs
DaniPopes 1551501
Merge branch 'main' into dani/signers
DaniPopes 15bee3b
update ledger tests
DaniPopes e58bcae
update ledger tests again
DaniPopes a36d51a
use serial
DaniPopes 4cdc706
fix
DaniPopes aded01f
todos
DaniPopes e11de38
Merge branch 'main' into dani/signers
DaniPopes e06356a
auto_impl
DaniPopes 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
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 +1 @@ | ||
msrv = "1.65" | ||
msrv = "1.68" |
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,30 @@ | ||
[package] | ||
name = "alloy-signer-aws" | ||
description = "Ethereum AWS KMS signer" | ||
|
||
version.workspace = true | ||
edition.workspace = true | ||
rust-version.workspace = true | ||
authors.workspace = true | ||
license.workspace = true | ||
homepage.workspace = true | ||
repository.workspace = true | ||
exclude.workspace = true | ||
|
||
[dependencies] | ||
alloy-primitives.workspace = true | ||
alloy-signer.workspace = true | ||
|
||
async-trait.workspace = true | ||
aws-sdk-kms = { version = "1.1", default-features = false } | ||
k256.workspace = true | ||
spki.workspace = true | ||
thiserror.workspace = true | ||
tracing.workspace = true | ||
|
||
[dev-dependencies] | ||
tokio = { workspace = true, features = ["macros", "rt-multi-thread"] } | ||
aws-config = { version = "1.0", default-features = false } | ||
|
||
[features] | ||
eip712 = ["alloy-signer/eip712"] |
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 |
---|---|---|
@@ -0,0 +1,5 @@ | ||
# alloy-signer-aws | ||
|
||
Ethereum [AWS KMS] signer. | ||
|
||
[AWS KMS]: https://aws.amazon.com/kms |
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 |
---|---|---|
@@ -0,0 +1,22 @@ | ||
#![doc = include_str!("../README.md")] | ||
#![doc( | ||
html_logo_url = "https://raw.githubusercontent.com/alloy-rs/core/main/assets/alloy.jpg", | ||
html_favicon_url = "https://raw.githubusercontent.com/alloy-rs/core/main/assets/favicon.ico" | ||
)] | ||
#![warn( | ||
missing_copy_implementations, | ||
missing_debug_implementations, | ||
missing_docs, | ||
unreachable_pub, | ||
clippy::missing_const_for_fn, | ||
rustdoc::all | ||
)] | ||
#![cfg_attr(not(test), warn(unused_crate_dependencies))] | ||
#![deny(unused_must_use, rust_2018_idioms)] | ||
#![cfg_attr(docsrs, feature(doc_cfg, doc_auto_cfg))] | ||
|
||
#[macro_use] | ||
extern crate tracing; | ||
|
||
mod signer; | ||
pub use signer::{AwsSigner, AwsSignerError}; |
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.
aws-sdk-kms 1.68, with msrv at 1.70 in readme. I think we'll want to bump this to latest stable for AFIT anyway