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

[P2P][Consensus] Separate Consensus and P2P Private Keys #750

Open
10 tasks
Tracked by #729
jessicadaugherty opened this issue May 14, 2023 · 1 comment
Open
10 tasks
Tracked by #729

[P2P][Consensus] Separate Consensus and P2P Private Keys #750

jessicadaugherty opened this issue May 14, 2023 · 1 comment
Assignees
Labels
consensus Consensus specific changes p2p P2P specific changes

Comments

@jessicadaugherty
Copy link
Contributor

Objective

Separate Consensus and P2P Private Keys to improve the security posture, enable greater flexibility in key management strategies, and isolate the impact of potential key compromises.

Origin Document

ADR-0004

Goals

  • Separate private keys for P2P and consensus modules
  • Implement a secure and efficient key management strategy for each module
  • Ensure the isolation of potential key compromises between the modules

Deliverable

  • Update the codebase to use separate private keys for P2P and consensus modules
  • Implement secure key management strategies for both modules
  • Update related documentation and code comments to reflect the separation of private keys for P2P and consensus modules

Non-goals / Non-deliverables

  • Refactoring of the codebase beyond the scope of implementing separate private keys for P2P and consensus modules

General issue deliverables

  • Update the appropriate CHANGELOG
  • Update any relevant READMEs (local and/or global)
  • Update any relevant global documentation & references
  • If applicable, update the source code tree explanation
  • If applicable, add or update a state, sequence or flowchart diagram using mermaid

Testing Methodology

  • All tests: make test_all
  • LocalNet: verify a LocalNet is still functioning correctly by following the instructions at docs/development/README.md

Creator: @jessicadaugherty

@jessicadaugherty jessicadaugherty converted this from a draft issue May 14, 2023
@jessicadaugherty jessicadaugherty added p2p P2P specific changes consensus Consensus specific changes labels May 14, 2023
@Olshansk Olshansk moved this to Rescope in V1 Dashboard May 15, 2023
@Olshansk
Copy link
Member

@bryanchriswhite I've left a few comments & questions in pokt-network/pocket-network-protocol#35, so let's resolve those and merge in the ADR to see if/how this should be scoped.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
consensus Consensus specific changes p2p P2P specific changes
Projects
Status: Rescope
Development

No branches or pull requests

3 participants