-
Notifications
You must be signed in to change notification settings - Fork 14
feat: axelar oracle #137
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
base: main
Are you sure you want to change the base?
feat: axelar oracle #137
Conversation
reednaa
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine to me.
Before we merge, we should consider a way to display which contracts (including oracles) have been audited and which have not.
| if (source == address(0)) revert InvalidAddress(); | ||
| if (payloads.length == 0) revert EmptyPayloadsNotAllowed(); | ||
|
|
||
| if (!IAttester(source).hasAttested(payloads)) revert NotAllPayloadsValid(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Opinion) Should be implemented in the outer submit function. The inner function is (mainly) concerned with messaging layer logic.
| function gateway() public view returns (IAxelarGateway) { | ||
| return IAxelarGateway(gatewayAddress); | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some strange code inherited from Axalar 😅
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Description
This PR implements an Axelar-based oracle for cross-chain message attestation.
Oracle Implementation:
Public Interface:
Design Decisions:
Related Issues
There is no related issue.
Third-Party Integration Checklist
foundry.tomlsrc/oracles/[oracle-type]/external/[protocol-name]/Additional Notes