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

Forgetting to run Contract::preflight results in difficult to debug error #224

Closed
nategraf opened this issue Sep 6, 2024 · 1 comment · Fixed by #239
Closed

Forgetting to run Contract::preflight results in difficult to debug error #224

nategraf opened this issue Sep 6, 2024 · 1 comment · Fixed by #239
Assignees
Labels

Comments

@nategraf
Copy link
Contributor

nategraf commented Sep 6, 2024

An application developer using Steel ran into an issue when using Steel when they forgot to run Contract::preflight on the host. It is expected that this will result in a failure. However the following assert was what caught the issue, and they had a very difficult time debugging it.

assert_eq!(self.header.state_root(), &state_root, "State root mismatch");

It is likely this will be a common error, and so we should make it easier to figure out how to fix it.

@nategraf nategraf added the bug Something isn't working label Sep 6, 2024
Copy link

linear bot commented Sep 6, 2024

@nategraf nategraf added steel and removed bug Something isn't working labels Sep 6, 2024
@Wollac Wollac self-assigned this Sep 11, 2024
Wollac added a commit that referenced this issue Sep 12, 2024
…239)

Returns "no accounts accessed: use Contract::preflight" when no
`Contract::preflight` was called.

closes #224 
closes WEB3-107
Wollac added a commit that referenced this issue Oct 7, 2024
…239)

Returns "no accounts accessed: use Contract::preflight" when no
`Contract::preflight` was called.

closes #224 
closes WEB3-107
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants