Skip to content
This repository was archived by the owner on Sep 14, 2025. It is now read-only.

feat: Integrate Aggregator in HelloWorldServiceManager - #85

Open
afkbyte wants to merge 5 commits into
masterfrom
afkbyte-avs-1063-modify-helloworld-contract-for-aggregator
Open

feat: Integrate Aggregator in HelloWorldServiceManager#85
afkbyte wants to merge 5 commits into
masterfrom
afkbyte-avs-1063-modify-helloworld-contract-for-aggregator

Conversation

@afkbyte

@afkbyte afkbyte commented Oct 9, 2024

Copy link
Copy Markdown

This PR contains only the changes to the HelloWorldServiceManager required to integrate with an aggregator service.

  • Added address public aggregator to store the aggregator's address
    • implemented onlyAggregator modifier to restrict task response
  • Modified respondToTask function to:
    • Accept aggregated signature data
    • Verify signatures using ECDSAStakeRegistry
    • Check if all signers are registered operators
    • Store aggregated response data
  • update deploy scripts

@afkbyte
afkbyte requested review from Sidu28 and stevennevins October 9, 2024 13:58
@wesfloyd

Copy link
Copy Markdown
Contributor

Reminder for Steven and Wes: might warrant a point release when ready.

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.

2 participants