Skip to content

fix: correct ecdsa message hash check #1563

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.

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

Merged
merged 3 commits into from
Jul 23, 2025

Conversation

0xrajath
Copy link
Collaborator

@0xrajath 0xrajath commented Jul 23, 2025

Motivation:

The ECDSA message hash check was being incorrectly calculated. It just needed to be keccak256(result).

@0xrajath 0xrajath self-assigned this Jul 23, 2025
@0xrajath 0xrajath added the ⌛ Hourglass Hourglass, a task-based AVS framework label Jul 23, 2025
@0xrajath 0xrajath marked this pull request as ready for review July 23, 2025 14:56
@0xrajath 0xrajath requested a review from ypatil12 July 23, 2025 14:58
@0xrajath 0xrajath merged commit db991de into release-dev/hourglass Jul 23, 2025
9 checks passed
@0xrajath 0xrajath deleted the rajath/fix-ecdsa-message-hash branch July 23, 2025 15:07
0xrajath added a commit that referenced this pull request Jul 24, 2025
**Motivation:**

The ECDSA message hash check was being incorrectly calculated. It just
needed to be `keccak256(result)`.
@0xrajath 0xrajath mentioned this pull request Jul 24, 2025
0xrajath added a commit that referenced this pull request Jul 25, 2025
**Motivation:**

The ECDSA message hash check was being incorrectly calculated. It just
needed to be `keccak256(result)`.
0xrajath added a commit that referenced this pull request Jul 30, 2025
**Motivation:**

The ECDSA message hash check was being incorrectly calculated. It just
needed to be `keccak256(result)`.
0xrajath added a commit that referenced this pull request Jul 30, 2025
# v1.8.0 Hourglass

The Hourglass release consists of a framework that supports the creation
of task-based AVSs. The task-based AVSs are enabled through a
`TaskMailbox` core contract deployed to all chains that support a
`CertificateVerifier`. Additionally AVSs deploy their
`TaskAVSRegistrar`. The release has 3 components:

1. Core Contracts
2. AVS Contracts
3. Offchain Infrastructure

The below release notes cover Core Contracts. For more information on
the end to end protocol, see our
[docs](https://github.com/Layr-Labs/hourglass-monorepo/blob/master/README.md)
and [core contract docs](../docs/avs/task/TaskMailbox.md).

## Release Manager

@0xrajath 

## Highlights

This hourglass release only introduces new contracts. As a result, there
are no breaking changes or deprecations.

🚀 New Features

Destination Chain Contracts
- `TaskMailbox`: A core infrastructure contract that enables task-based
AVS execution models. It provides a standardized way for AVSs to create
tasks, have operators execute them, and submit verified results
on-chain. The contract acts as a mailbox system where task creators post
tasks with fees, and operators compete to execute and submit results
with proper consensus verification.

## Changelog

- feat: no consensus config [PR
#1574](#1574)
- fix: hourglass internal review fixes [PR
#1570](#1570)
- fix: addressing pr comments [PR
#1568](#1568)
- fix: deploy script
- fix: deploy script
- fix: hourglass release upgrade semver
- fix: hourglass release upgrade semver
- docs: changelog
- fix: correct ecdsa message hash check [PR
#1563](#1563)
- fix: missing assume in fuzz test
- fix: `submitResult` certificate checks [PR
#1557](#1557)
- chore: forge fmt
- fix: certificate verifier interface changes
- feat: hourglass zeus script [PR
#1546](#1546)
- fix: mock certificate verifiers [PR
#1545](#1545)
- feat: hourglass (task-based AVS framework) [PR
#1534](#1534)

---------

Co-authored-by: Nadir Akhtar <[email protected]>
@0xrajath 0xrajath mentioned this pull request Jul 30, 2025
0xrajath added a commit that referenced this pull request Jul 30, 2025
# v1.8.0 Hourglass

The Hourglass release consists of a framework that supports the creation
of task-based AVSs. The task-based AVSs are enabled through a
`TaskMailbox` core contract deployed to all chains that support a
`CertificateVerifier`. Additionally AVSs deploy their
`TaskAVSRegistrar`. The release has 3 components:

1. Core Contracts
2. AVS Contracts
3. Offchain Infrastructure

The below release notes cover Core Contracts. For more information on
the end to end protocol, see our
[docs](https://github.com/Layr-Labs/hourglass-monorepo/blob/master/README.md)
and [core contract docs](../docs/avs/task/TaskMailbox.md).

## Release Manager

@0xrajath 

## Highlights

This hourglass release only introduces new contracts. As a result, there
are no breaking changes or deprecations.

🚀 New Features

Destination Chain Contracts
- `TaskMailbox`: A core infrastructure contract that enables task-based
AVS execution models. It provides a standardized way for AVSs to create
tasks, have operators execute them, and submit verified results
on-chain. The contract acts as a mailbox system where task creators post
tasks with fees, and operators compete to execute and submit results
with proper consensus verification.

## Changelog

- feat: no consensus config [PR
#1574](#1574)
- fix: hourglass internal review fixes [PR
#1570](#1570)
- fix: addressing pr comments [PR
#1568](#1568)
- fix: deploy script
- fix: deploy script
- fix: hourglass release upgrade semver
- fix: hourglass release upgrade semver
- docs: changelog
- fix: correct ecdsa message hash check [PR
#1563](#1563)
- fix: missing assume in fuzz test
- fix: `submitResult` certificate checks [PR
#1557](#1557)
- chore: forge fmt
- fix: certificate verifier interface changes
- feat: hourglass zeus script [PR
#1546](#1546)
- fix: mock certificate verifiers [PR
#1545](#1545)
- feat: hourglass (task-based AVS framework) [PR
#1534](#1534)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⌛ Hourglass Hourglass, a task-based AVS framework
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants