Skip to content

fix: handle error being thrown in shutdown of run#2595

Merged
matangeorgi merged 2 commits intomasterfrom
matan/handle-error-in-run
Nov 27, 2024
Merged

fix: handle error being thrown in shutdown of run#2595
matangeorgi merged 2 commits intomasterfrom
matan/handle-error-in-run

Conversation

@matangeorgi
Copy link
Copy Markdown
Contributor

@matangeorgi matangeorgi commented Nov 18, 2024

when the catch scope is being triggered in the run function, the catch will execute:

await this.shutdown();

If shutdown fails for some reason we have no information of what happened.
Adding an extra catch is for adding the missing information.

@matangeorgi matangeorgi requested a review from AviVahl November 18, 2024 16:36
Copy link
Copy Markdown
Contributor

@AviVahl AviVahl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test please

@barak007 barak007 requested a review from AviVahl November 19, 2024 13:46
@matangeorgi matangeorgi merged commit 4eef77d into master Nov 27, 2024
@matangeorgi matangeorgi deleted the matan/handle-error-in-run branch November 27, 2024 11:03
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