search

JCCDex / ProjectFundingProposal

Jingtum & MOAC Project Funding Proposal
28 stars 8 forks source link

PFP01 swtclib维护更新 #32

Closed lospringliu closed 4 years ago

lospringliu commented 5 years ago

五月份完成 swtc-lib 后， 基本上每个月都要花一两天更新。 其实还是有很多可以提高的地方，比如发布esm格式库， 测试永远有改进空间

日常是需要在功能上赶上官方jingtum-lib的脚步， 主要是 收费挂单和多方签名； 另外也接收了社区的贡献包括井畅的serialier的typescript化， Weijia的ipfs相关的更改 以及A计划（未批准）相关的实现

目前出来一个比较大的变动，多方签名， 需要静下来花些精力了。除了支持多签，这次会顺带把serializer中不一致的常量和官方统一。

工作量预估

lospringliu commented 5 years ago

文档： https://swtcdoc.netlify.com/docs/swtclib/#multiSign 源码： https://github.com/swtcca/swtclib/commit/4c8251e411912a6e5f97db47e25995730d28d033 类型定义 和 测试 教程更新： https://swtclearn.netlify.com/ 100%

GinMu commented 4 years ago

swtc-transaction多签名审计

  1. 版本: 1.2.26
  2. 审计人: GinMu
  3. 审查方法: 代码review和在测试链上实际测试, 测试代码swtclib-test
  4. 审计内容:
    1. 设置多签列表，查询多签列表信息和提交多签交易
  5. 审计结果:
    1. 在测试链上设置多签列表，查询多签列表信息和提交多签交易成功
    2. 单元测试已覆盖
    3. 文档已更新
  6. 建议:
    1. ISignerListTxOptions lists最好不要定义成any[], 可选参数address并未实际使用
    2. 针对多签数据组装和验签可以抽取出来，作为单独的工具库，方便第三方直接调用
    3. fee校验存疑，实际测试来看，fee设置为10同样可以提交成功
  7. 结果: 通过
Xiaohao204 commented 4 years ago

swtc-transaction多重签名审计

  1. 版本: 1.2.26
  2. 审计人: xiaohao
  3. 审查方法: 代码review和在正式链上实际测试, 测试代码swtclib-test
  4. 审计内容:
    1. 设置多签列表，查询多签列表信息，提交多签交易，废弃主账号密钥和激活主账户密钥
  5. 审计结果:
    1. 在测试链上设置多签列表，查询多签列表信息，提交多签交易成功，废弃主账号密钥和激活主账户密钥
    2. 单元测试已覆盖
    3. 文档已更新
  6. 建议: 针对多签数据组装和验签可以抽取出来，作为单独的工具库，方便第三方直接调用
  7. 结果: 通过
lospringliu commented 4 years ago

@GinMu , 费用最终取决于区块链底层 合约因为只有ts5环境，好像现在跑不通， 不过不值得立刻去排查，等成熟后再讲

zhaitianye commented 4 years ago

swtc-transaction多重签名审计

  1. 版本: 1.2.26
  2. 审计人: 翟天野
  3. 审查方法: 代码review和在正式链上实际测试, 测试代码test_swtc_lib_proxy
  4. 审计内容:
    1. 设置多签列表
    2. 查询多签列表信息
    3. 提交多签交易方式1
    4. 提交多签交易方式2
    5. 废弃主账号密钥
    6. 激活主账户密钥
  5. 审计结果:
    1. 在正式链上设置多签列表
    2. 查询多签列表信息
    3. 提交多签交易成功
    4. 废弃主账号密钥
    5. 激活主账户密钥
    6. 单元测试已覆盖
    7. 文档已更新
  6. 建议:
    1. 设置帐号的签名列表，buildSignerListTx在传入参数时候，是否应该和rpc上的一致，使用 SignerQuorum和SignerEntries变量命名，降低开发者疑惑
    2. 需要设置足够的燃料支持多签交易，这个燃料什么时候会用到，需要用多少，最低和建议设置多少，具体场景应做详细说明
    3. 激活主密钥的时候需要完整的列表还是只需要账号阈值大于设置的阈值即可，需要在文档里面做出说明
    4. 是否有一个接口来查询废除主秘钥之后的状态。
    5. 设置账号的多签列表之后，此账号为多签账号，可否有方法把此账号设置回来，即设置为普通账号
    6. 批量签名的签名列表是2-8个，这个是否应该在文档里面进行说明，超过8个会提示失败
  7. 结果: 通过
Xiaohao204 commented 4 years ago

j3zEdnLUMQ6rXsKk8zNCotdNNfh4avFttK

zhaitianye commented 4 years ago

jEd4Vmp9Hr3d6snwcSsBy6Gt25kzkHLQWt

GinMu commented 4 years ago

jwnqKpXJYJPeAnUdVUv3LfbxiJh5ZVXh79

AnFengDe commented 4 years ago

付费记录 审计评估是3个工作日，有3人提交，为鼓励大家，发三份审计费用 @Xiaohao204 付费记录 @zhaitianye 付费记录 @GinMu 付费记录