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.

Sign up for GitHub

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

Jump to bottom

[venus] 钱包增加删除地址功能 #5252

Closed
1 of 11 tasks
Tracked by #5296
cloudxin opened this issue Sep 1, 2022 · 1 comment · Fixed by #5326
Closed
1 of 11 tasks
Tracked by #5296

[venus] 钱包增加删除地址功能 #5252

cloudxin opened this issue Sep 1, 2022 · 1 comment · Fixed by #5326
Assignees
@hunjixin
Labels
C-enhancement Category: feature enhancement C-need-testing Category: Venus testing related CU-force-community Category: from force community

Comments

@cloudxin
Copy link

cloudxin commented Sep 1, 2022

链服务模块 / Chain Service Components

  • venus
  • venus-auth
  • venus-gateway
  • venus-messager
  • venus-miner
  • 文档 / docs

订单服务模块 / Deal Service Components

  • venus-market
  • 文档 / docs

算力服务模块 / Storage Power Service Components

  • venus-sector-manager
  • venus-worker
  • 文档 / docs

描述 / Description

venus在导入私钥，或者创建钱包地址后，节点上的地址信息无法删除；
这样解决从单venus节点签名消息，切换到venus链服务后产生签名消息混乱的问题。
增加一下假删除方法，禁用某个地址在venus节点上的使用。而统一由venus-gateway来签名消息
@cloudxin cloudxin added the C-enhancement Category: feature enhancement label Sep 1, 2022
@Fatman13 Fatman13 added the C-triage Category: Need triage label Sep 2, 2022
@Fatman13 Fatman13 added the CU-force-community Category: from force community label Sep 2, 2022
@zl03jsj
Copy link
Contributor

zl03jsj commented Sep 5, 2022

应该在venus中支持删除私钥的方法, 但为了安全性, 应该在删除前对私钥(钱包)的密码进行校验.

@zl03jsj zl03jsj removed the C-triage Category: Need triage label Sep 5, 2022
@zl03jsj zl03jsj removed their assignment Sep 5, 2022
@hunjixin hunjixin changed the title [venus] 钱包增加删除或禁用功能 [venus] 钱包增加删除地址功能 Sep 17, 2022
@diwufeiwen diwufeiwen mentioned this issue Sep 19, 2022
Closed
54 tasks
@diwufeiwen diwufeiwen mentioned this issue Sep 26, 2022
Merged
5 tasks
@simlecode simlecode moved this to Done in Venus Project Sep 26, 2022
@Fatman13 Fatman13 added the C-need-testing Category: Venus testing related label Oct 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hunjixin hunjixin

Labels
C-enhancement Category: feature enhancement C-need-testing Category: Venus testing related CU-force-community Category: from force community
Projects
Venus Project
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add delete wallet address cmd / 删除钱包命令 filecoin-project/venus
4 participants
@Fatman13 @zl03jsj @hunjixin @cloudxin