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.

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

Prevent DumpOnHealthCheckRestart from crashing the monitor #1609

Merged
merged 1 commit into from
Jul 23, 2023

Conversation

Cyberboss
Copy link
Member

🆑
Fixed a watchdog crash that could occur if dumpOnHealthCheckRestart was enabled and the DreamDaemon process aborts.
/:cl:

@Cyberboss Cyberboss added Fix Fixes incorrect functionality Area: Watchdog With regard to managing DreamDaemon servers labels Jul 23, 2023
@Cyberboss Cyberboss added this to the v5.13.5 milestone Jul 23, 2023
@Cyberboss Cyberboss enabled auto-merge July 23, 2023 14:56
@codecov
Copy link

codecov bot commented Jul 23, 2023

Codecov Report

Merging #1609 (5b17368) into master (7ee807f) will decrease coverage by 0.02%.
The diff coverage is 27.27%.

@@            Coverage Diff             @@
##           master    #1609      +/-   ##
==========================================
- Coverage   95.01%   95.00%   -0.02%     
==========================================
  Files         606      606              
  Lines      126336   126346      +10     
==========================================
- Hits       120044   120038       -6     
- Misses       6292     6308      +16     

@Cyberboss Cyberboss merged commit fee685d into master Jul 23, 2023
@Cyberboss Cyberboss deleted the DumpOnHealthCheckRestartFix branch July 23, 2023 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Watchdog With regard to managing DreamDaemon servers Fix Fixes incorrect functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant