Skip to content
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

contracts: add support for core.CallDataPublicKey and core.CurrentEpoch #370

Merged
merged 1 commit into from
Aug 30, 2024

Conversation

CedarMist
Copy link
Member

Adds two methods to Subcall.sol

  • coreCallDataPublicKey
  • coreCurrentEpoch

@CedarMist CedarMist self-assigned this Aug 29, 2024
@CedarMist CedarMist added the contracts Pull requests that update sapphire-contracts label Aug 29, 2024
Copy link

netlify bot commented Aug 29, 2024

Deploy Preview for oasisprotocol-sapphire-paratime canceled.

Name Link
🔨 Latest commit cc01fde
🔍 Latest deploy log https://app.netlify.com/sites/oasisprotocol-sapphire-paratime/deploys/66d1f518d89cc90008d10b66

@CedarMist CedarMist marked this pull request as ready for review August 29, 2024 16:44
@CedarMist CedarMist requested review from matevz and lubej August 29, 2024 16:44
Copy link
Member

@matevz matevz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job!

For production networks we should add a Web3 client monitoring Testnet and Mainnet call data public keys here: https://github.com/oasisprotocol/network-monitoring/tree/main/web3

contracts/test/subcall.ts Show resolved Hide resolved
@CedarMist
Copy link
Member Author

Good job!

For production networks we should add a Web3 client monitoring Testnet and Mainnet call data public keys here: https://github.com/oasisprotocol/network-monitoring/tree/main/web3

I'll be using the subcall (directly, not via the contract) in the go, python & js clients, so if encrypted transactions are being performed it will have validated the calldata public key and successfully used it.

@CedarMist CedarMist merged commit 1bb637b into main Aug 30, 2024
10 checks passed
@CedarMist CedarMist deleted the CedarMist/cdpk_subcall branch August 30, 2024 16:50
github-actions bot added a commit that referenced this pull request Aug 30, 2024
…edarMist/cdpk_subcall

contracts: add support for core.CallDataPublicKey and core.CurrentEpoch 1bb637b
@CedarMist CedarMist added this to the contracts-0.2.11 milestone Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contracts Pull requests that update sapphire-contracts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants