Skip to content

Commit

Permalink
Archive remote_signer code (#2559)
Browse files Browse the repository at this point in the history
## Proposed Changes

This PR deletes all `remote_signer` code from Lighthouse, for the following reasons:

* The `remote_signer` code is unused, and we have no plans to use it now that we're moving to supporting the Web3Signer APIs: #2522
* It represents a significant maintenance burden. The HTTP API tests have been prone to platform-specific failures, and breakages due to dependency upgrades, e.g. #2400.

Although the code is deleted it remains in the Git history should we ever want to recover it. For ease of reference:

- The last commit containing remote signer code: 5a3bcd2
- The last Lighthouse version: v1.5.1
  • Loading branch information
michaelsproul committed Sep 3, 2021
1 parent d9910f9 commit f4aa1d8
Show file tree
Hide file tree
Showing 50 changed files with 0 additions and 4,958 deletions.
396 changes: 0 additions & 396 deletions Cargo.lock

Large diffs are not rendered by default.

6 changes: 0 additions & 6 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@ members = [
"common/logging",
"common/lru_cache",
"common/malloc_utils",
"common/remote_signer_consumer",
"common/sensitive_url",
"common/slot_clock",
"common/task_executor",
Expand Down Expand Up @@ -69,17 +68,12 @@ members = [
"lighthouse",
"lighthouse/environment",

"remote_signer",
"remote_signer/backend",
"remote_signer/client",

"slasher",
"slasher/service",

"testing/ef_tests",
"testing/eth1_test_rig",
"testing/node_test_rig",
"testing/remote_signer_test",
"testing/simulator",
"testing/state_transition_vectors",

Expand Down
16 changes: 0 additions & 16 deletions common/remote_signer_consumer/Cargo.toml

This file was deleted.

89 changes: 0 additions & 89 deletions common/remote_signer_consumer/src/http_client.rs

This file was deleted.

213 changes: 0 additions & 213 deletions common/remote_signer_consumer/src/lib.rs

This file was deleted.

Loading

0 comments on commit f4aa1d8

Please sign in to comment.