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

Remote wallet management for lotus-miner #7670

Open
4 of 14 tasks
jennijuju opened this issue Nov 23, 2021 · 3 comments · May be fixed by #8428
Open
4 of 14 tasks

Remote wallet management for lotus-miner #7670

jennijuju opened this issue Nov 23, 2021 · 3 comments · May be fixed by #8428
Labels
area/wallet Area: Wallet good first issue Good for newcomers kind/feature Kind: Feature LM-tech-debt P2 P2: Should be resolved
Milestone

Comments

@jennijuju
Copy link
Member

Checklist

  • This is not a new feature or an enhancement to the Filecoin protocol. If it is, please open an FIP issue.
  • This is not brainstorming ideas. If you have an idea you'd like to discuss, please open a new discussion on the lotus forum and select the category as Ideas.
  • I have a specific, actionable, and well motivated feature request to propose.

Lotus component

  • lotus daemon - chain sync
  • lotus miner - mining and block production
  • lotus miner/worker - sealing
  • lotus miner - proving(WindowPoSt)
  • lotus miner/market - storage deal
  • lotus miner/market - retrieval deal
  • lotus miner/market - data transfer
  • lotus client
  • lotus JSON-RPC API
  • lotus message management (mpool)
  • Other

What is the motivation behind this feature request? Is your feature request related to a problem? Please describe.

to provide more flexibility to access miner box for operations without concerning about leaking the wallet keys.

Describe the solution you'd like

  • wallet being hosted on one machine
  • lotus miner has api access to the wallet node via a token
  • one can define the perm of the api token, i.e: a set of API that token is granted permission

Describe alternatives you've considered

No response

Additional context

No response

@jennijuju jennijuju added kind/feature Kind: Feature P2 P2: Should be resolved area/wallet Area: Wallet labels Nov 23, 2021
@jennijuju jennijuju added this to the v1.13.3 milestone Nov 23, 2021
@stuberman
Copy link

We understand that the private key is passed in the clear with API calls, please encrypt the data transfers. For a remote wallet this is extremely important.

@jennijuju
Copy link
Member Author

i dont think we should ever pass pk in api param - thats sounds like a terrible idea and def should avoid.

@travisperson
Copy link
Contributor

I messed around bit with this a while back for a few nights.

https://github.com/travisperson/secure-wallet

@jennijuju jennijuju modified the milestones: v1.13.3, v1.15.0 Jan 2, 2022
@jennijuju jennijuju linked a pull request Apr 4, 2022 that will close this issue
9 tasks
@rjan90 rjan90 added this to the LM-Tech-Debt milestone Mar 31, 2023
@ZenGround0 ZenGround0 added good first issue Good for newcomers and removed LotusY-1 labels Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/wallet Area: Wallet good first issue Good for newcomers kind/feature Kind: Feature LM-tech-debt P2 P2: Should be resolved
Projects
Status: No status
Status: Ready To Work
Development

Successfully merging a pull request may close this issue.

6 participants