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.

Sign up for GitHub

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

Jump to bottom

Plugin unhandled exception #3349

Merged
merged 6 commits into from
Jun 21, 2024
Merged

Plugin unhandled exception #3349

merged 6 commits into from
Jun 21, 2024

Commits on Jun 16, 2024

  1. [Neo Core] Part 1. Isolate Plugins Exceptions from the Node. (#3309) 

    * catch plugin exceptions.

* add UT test

* udpate format

* make the test  more complete

* complete the ut test

* format

* complete UT tests with NonPlugin case

* async invoke

* Update src/Neo/Ledger/Blockchain.cs

Co-authored-by: Christopher Schuchardt <cschuchardt88@gmail.com>

---------

Co-authored-by: Christopher Schuchardt <cschuchardt88@gmail.com>
    @Jim8y @cschuchardt88
    Jim8y and cschuchardt88 authored Jun 16, 2024
    Configuration menu
    Copy the full SHA
    89d6ea3 View commit details
    Browse the repository at this point in the history

Commits on Jun 18, 2024

  1. [Neo Plugin New feature] UnhandledExceptionPolicy on Plugin Unhandled… 

    … Exception (#3311)

* catch plugin exceptions.

* add UT test

* udpate format

* make the test  more complete

* complete the ut test

* format

* complete UT tests with NonPlugin case

* async invoke

* stop plugin on exception

* remove watcher from blockchain if uint test is done to avoid cross test data pollution.

* add missing file

* 3 different policy on handling plugin exception

* add missing file

* fix null warning

* format

* Apply suggestions from code review

Clean

* Update src/Neo/Plugins/PluginSettings.cs

Co-authored-by: Shargon <shargon@gmail.com>

* Update src/Neo/Plugins/PluginSettings.cs

Co-authored-by: Christopher Schuchardt <cschuchardt88@gmail.com>

* Update src/Plugins/TokensTracker/TokensTracker.cs

Co-authored-by: Christopher Schuchardt <cschuchardt88@gmail.com>

* Update src/Plugins/TokensTracker/TokensTracker.json

---------

Co-authored-by: Shargon <shargon@gmail.com>
Co-authored-by: Christopher Schuchardt <cschuchardt88@gmail.com>
    @Jim8y @shargon @cschuchardt88
    3 people authored Jun 18, 2024
    Configuration menu
    Copy the full SHA
    c696cec View commit details
    Browse the repository at this point in the history

Commits on Jun 19, 2024

  1. Merge branch 'master' into plugin-unhandled-exception

    @Jim8y
    Jim8y authored Jun 19, 2024
    Configuration menu
    Copy the full SHA
    045a7d8 View commit details
    Browse the repository at this point in the history

Commits on Jun 20, 2024

  1. make the exception message clear

    @Jim8y
    Jim8y committed Jun 20, 2024
    Configuration menu
    Copy the full SHA
    85d8a48 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' into plugin-unhandled-exception

    @shargon
    shargon authored Jun 20, 2024
    Configuration menu
    Copy the full SHA
    b763ec3 View commit details
    Browse the repository at this point in the history

Commits on Jun 21, 2024

  1. Merge branch 'master' into plugin-unhandled-exception

    @NGDAdmin
    NGDAdmin authored Jun 21, 2024
    Configuration menu
    Copy the full SHA
    2ae6dc3 View commit details
    Browse the repository at this point in the history