Skip to content
This repository has been archived by the owner on Sep 13, 2022. It is now read-only.

fix(proof): Verify aggregated signature in checking proof #308

Merged
merged 4 commits into from
Jun 16, 2020

Conversation

KaoImin
Copy link

@KaoImin KaoImin commented Jun 3, 2020

What type of PR is this?
fix

What this PR does / why we need it:

  1. Bump ophelia-bls-amcl version from 0.1 to 0.2 which includes an API argument change.
  2. Fix verify an aggregated signature in checking proof. The BLS public keys should be recovered from metadata, rather than the current Address-PubKey Map.

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

@muta-bot
Copy link

muta-bot bot commented Jun 3, 2020

Accept request.

@muta-bot
Copy link

muta-bot bot commented Jun 3, 2020

Docker builded. "mutadev/muta:d9e114c"
Run chaos test on k8s named "muta-pr308"

@muta-bot
Copy link

muta-bot bot commented Jun 4, 2020

Accept request.

@muta-bot
Copy link

muta-bot bot commented Jun 4, 2020

Docker builded. "mutadev/muta:2cac905"
Run chaos test on k8s named "muta-pr308"

@muta-bot
Copy link

muta-bot bot commented Jun 4, 2020

Accept request.

@muta-bot
Copy link

muta-bot bot commented Jun 4, 2020

Docker builded. "mutadev/muta:312c120"
Run chaos test on k8s named "muta-pr308"

@KaoImin KaoImin requested review from LycrusHamster and yejiayu June 4, 2020 03:45
@muta-bot
Copy link

muta-bot bot commented Jun 4, 2020

Accept request.

@muta-bot
Copy link

muta-bot bot commented Jun 4, 2020

Docker builded. "mutadev/muta:734106b"
Run chaos test on k8s named "muta-pr308"

@muta-bot
Copy link

muta-bot bot commented Jun 4, 2020

Date(muta-pr308) 1 2 3 4
2020-06-04T06:41:47.448Z 0 0 0 0
2020-06-04T06:45:47.504Z 0 0 0 0
2020-06-04T06:49:47.560Z 0 0 0 0
2020-06-04T06:53:48.600Z 0 0 0 0
2020-06-04T06:57:49.640Z 0 0 0 0
2020-06-04T07:01:51.208Z 0 0 0 0
2020-06-04T07:05:51.240Z 0 0 0 0
2020-06-04T07:09:51.276Z 0 0 0 0
2020-06-04T07:13:51.308Z 0 0 0 0
2020-06-04T07:17:51.340Z
2020-06-04T07:21:51.348Z
2020-06-04T07:25:51.376Z
2020-06-04T07:29:51.384Z
2020-06-04T07:33:51.392Z
2020-06-04T07:37:51.400Z
2020-06-04T07:41:51.412Z
2020-06-04T07:45:51.444Z
2020-06-04T07:49:51.452Z
2020-06-04T07:53:51.460Z
2020-06-04T07:57:51.472Z
2020-06-04T08:01:51.484Z
2020-06-04T08:05:51.520Z
2020-06-04T08:09:51.528Z
2020-06-04T08:13:51.536Z
tx_block sec_block tx_sec avg_round
undefined undefined undefined undefined

@yejiayu yejiayu changed the title fix(proof): verify aggregated signature in checking proof fix(proof): Verify aggregated signature in checking proof Jun 5, 2020
@yejiayu yejiayu merged commit d2a98b0 into master Jun 16, 2020
@yejiayu yejiayu deleted the fix-change-list branch July 8, 2020 07:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants