Skip to content

halt the code execution when metrics server fails to start#133

Merged
WorldDogs merged 1 commit intonode_auditfrom
fix_node/audit_n9
Mar 21, 2024
Merged

halt the code execution when metrics server fails to start#133
WorldDogs merged 1 commit intonode_auditfrom
fix_node/audit_n9

Conversation

@FletcherMan
Copy link
Copy Markdown
Collaborator

No description provided.

@FletcherMan FletcherMan requested a review from WorldDogs March 18, 2024 09:49
@FletcherMan FletcherMan mentioned this pull request Mar 18, 2024
10 tasks
@WorldDogs WorldDogs merged commit b752cce into node_audit Mar 21, 2024
@WorldDogs WorldDogs deleted the fix_node/audit_n9 branch March 21, 2024 01:16
FletcherMan added a commit that referenced this pull request Apr 1, 2024
* handle unhandled errors (#123)

Co-authored-by: fletcher.fan <fletcher.fan@fletcher2fan.local>

* Declaration redundancy (#126)

Co-authored-by: fletcher.fan <fletcher.fan@fletcher2fan.local>

* fix errors comparsion (#128)

Co-authored-by: fletcher.fan <fletcher.fan@fletcher2fan.local>

* unified function receiver (#129)

Co-authored-by: fletcher.fan <fletcher.fan@fletcher2fan.local>

* change to not use reserved words for variables (#130)

Co-authored-by: fletcher.fan <fletcher.fan@fletcher2fan.local>

* Fix potential single point of failure when interacting with l2 contracts (#136)

* halt the code execution when metrics server fails to start (#133)

Co-authored-by: fletcher.fan <fletcher.fan@fletcher2fan.local>

* upgrade the dependency version (#125)

Co-authored-by: fletcher.fan <fletcher.fan@fletcher2fan.local>

* consider wrapped errors on type assertion on errors (#131)

Co-authored-by: fletcher.fan <fletcher.fan@fletcher2fan.local>

* create copy of variables used in loop (#132)

Co-authored-by: fletcher.fan <fletcher.fan@fletcher2fan.local>

* upgrade go-ethereum/tendermint dependency

---------

Co-authored-by: fletcher.fan <fletcher.fan@fletcher2fan.local>
Co-authored-by: WorldDogs <33647825+WorldDogs@users.noreply.github.com>
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.

2 participants