Skip to content

Improve error handling for HTTP status code 0 errors; add blockNumber to Block Trigger output #217

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 10 commits into from
Apr 21, 2025

Conversation

chrisli30
Copy link
Member

No description provided.

devin-ai-integration bot and others added 10 commits April 5, 2025 00:44
* Fix code formatting with gofmt

Co-Authored-By: Chris Li <chris@avaprotocol.org>

* Fix nil pointer dereference in ContractWriteProcessor.Execute

Co-Authored-By: Chris Li <chris@avaprotocol.org>

* Revert changes to vm_runner_contract_write.go to wait for PR #199

Co-Authored-By: Chris Li <chris@avaprotocol.org>

---------

Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: Chris Li <chris@avaprotocol.org>
* Improve error handling for HTTP status code 0 errors

Co-Authored-By: Chris Li <chris@avaprotocol.org>

* Add unit test for HTTP error handling and fix error message capitalization

Co-Authored-By: Chris Li <chris@avaprotocol.org>

* Fix unused import in vm_runner_rest_test.go

Co-Authored-By: Chris Li <chris@avaprotocol.org>

* Updated the testing instructions in README.md

* Fixed test errors from TestRestRequestErrorHandling

* Added Paymaster contract address to docs/contract.md

* Added fixes for TestContractWriteSimpleReturn case

* Updated error message in operator/operator.go

* Added missing txReceipt fields to vm_runner_contract_write.go and test their existence.

* Updated vm_runner_rest.go to handle status 0

* Revert the capitalization of Errorf messages

---------

Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: Chris Li <chris@avaprotocol.org>
Co-authored-by: chrisli30 <chrisli30@users.noreply.github.com>
…o ms (#207)

* Add backup ability to run with repl console

* add a migration framework, create second -> ms converted

* autoformat code on push

* Use go1.24 in Dockerfile

* fix new grpc format

* add new packages to test

* add new pkg to test run

* fix AI slop

* fix timestamp change

---------

Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
…ds (#208)

* docs: update development.md with improved formatting and REPL command documentation

Co-Authored-By: vinh@avaprotocol.org <vinh@avaprotocol.org>

* docs: add missing rm and backup commands to development.md

Co-Authored-By: vinh@avaprotocol.org <vinh@avaprotocol.org>

* docs: update rm command example to use generic prefix

Co-Authored-By: vinh@avaprotocol.org <vinh@avaprotocol.org>

---------

Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: vinh@avaprotocol.org <vinh@avaprotocol.org>
@chrisli30 chrisli30 merged commit 105835b into main Apr 21, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants