Skip to content

Catch all exceptions from Exit shutdown #2079

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 2 commits into from
Jul 26, 2019

Conversation

jkotalik
Copy link

Same as dotnet/aspnetcore#12518 except in master and with generic host.

TODO I think I can test this one with a bad logger. I'll take a stab at it.

@jkotalik jkotalik requested a review from Tratcher July 24, 2019 18:39
@jkotalik jkotalik requested a review from analogrelay as a code owner July 24, 2019 18:39
@jkotalik jkotalik force-pushed the jkotalik/genericHostConsoleException branch from f5dfebd to 139b335 Compare July 25, 2019 15:56
@jkotalik
Copy link
Author

🆙 📅 . I could hack together a test where I replace the IConsole, but I would need to make _messageQueue internal and do a bit of other magic.

@jkotalik jkotalik requested a review from BrennanConroy July 26, 2019 14:59
@jkotalik jkotalik merged commit 42d979d into master Jul 26, 2019
@ghost ghost deleted the jkotalik/genericHostConsoleException branch July 26, 2019 15:59
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Feb 28, 2020
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Mar 2, 2020
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Mar 11, 2020
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Mar 27, 2020
@ghost ghost locked as resolved and limited conversation to collaborators May 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants