Skip to content

Commit

Permalink
Signature Aggregation Refactor (#883)
Browse files Browse the repository at this point in the history
* remove unused code

* remove signature aggregation job

* appease linter

* remove signature aggregation job

* move all signature aggregeation logic to AggregateSignatures

* nit

* nit

* nit

* add mock and tests

* clean up tests

* nit

* don't autogen SignatureGetter because it requires manual changes because gomock is broken

* nit

* comment

* comments

* lower log level

* re-add todo

* nit

* refactor aggregeateSignatures

* comment nits

* add tests

* use error from avalancheWap; combine functions; nits

* add logs

* typo fix
  • Loading branch information
Dan Laine authored Sep 29, 2023
1 parent b8c0dfd commit 40d1217
Show file tree
Hide file tree
Showing 9 changed files with 654 additions and 745 deletions.
2 changes: 1 addition & 1 deletion plugin/evm/vm.go
Original file line number Diff line number Diff line change
Expand Up @@ -943,7 +943,7 @@ func (vm *VM) CreateHandlers(context.Context) (map[string]*commonEng.HTTPHandler
}

if vm.config.WarpAPIEnabled {
warpAggregator := aggregator.NewAggregator(vm.ctx.SubnetID, warpValidators.NewState(vm.ctx), &aggregator.NetworkSigner{Client: vm.client})
warpAggregator := aggregator.New(vm.ctx.SubnetID, warpValidators.NewState(vm.ctx), &aggregator.NetworkSigner{Client: vm.client})
if err := handler.RegisterName("warp", warp.NewAPI(vm.warpBackend, warpAggregator)); err != nil {
return nil, err
}
Expand Down
170 changes: 0 additions & 170 deletions warp/aggregator/aggregation_job.go

This file was deleted.

Loading

0 comments on commit 40d1217

Please sign in to comment.