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

Inter contract call throws Revert(123) #2819

Closed
danielbate opened this issue Jul 23, 2024 · 1 comment
Closed

Inter contract call throws Revert(123) #2819

danielbate opened this issue Jul 23, 2024 · 1 comment
Assignees
Labels
bug Issue is a bug

Comments

@danielbate
Copy link
Member

Cross referenced with the sway issue

One of our ecosystem projects is reporting problems when trying to create a dApp where the contract includes an inter contract call to another contract.

The error suggests its a mismatched selector, however certain functions from the same contract, with the same ABI work successfully.

I've created a minimal reproduction that demonstrates this behaviour in both TS and RS. However, it has been reported that the RS test does work successfully locally, currently I have it running against testnet.

First step would be to port the RS test to local to get it working successfully, then we can compare what is happening between the 2 SDKs.

@danielbate danielbate added the p1 label Jul 23, 2024
@arboleya arboleya added bug Issue is a bug p1 and removed p1 labels Jul 23, 2024
@danielbate danielbate self-assigned this Jul 24, 2024
@danielbate
Copy link
Member Author

@petertonysmith94 diligently spotted that upon upgrading Pyth to the latest versions of forc@0.62.0 and fuel-core@0.31.0 and the SDK 0.92.1, this problem was resolved. I think the main culprit was this Sway change. We've gone back and fourth with them over version changes, this has consistently been the reason for problems and we have not had to make any changes to the SDK, no bugs have been found.

I've also offered a new working reproduction and integrated local TS testing with their own exemplar repository to better support them.

Closing this issue as this is not a problem with the SDK.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue is a bug
Projects
None yet
Development

No branches or pull requests

3 participants