Skip to content

Add a shutdown handler to the certifier plugin#9795

Closed
bneradt wants to merge 1 commit intoapache:masterfrom
bneradt:fix_certifier_shutdown_crash
Closed

Add a shutdown handler to the certifier plugin#9795
bneradt wants to merge 1 commit intoapache:masterfrom
bneradt:fix_certifier_shutdown_crash

Conversation

@bneradt
Copy link
Copy Markdown
Contributor

@bneradt bneradt commented Jun 7, 2023

This adds a shutdown handler to the certifier plugin so that its SSL data can be cleaned up in a callback rather than on process shutdown.

This also reverts commenting out the debug call from SslData which was commented out as a workaround to a crash which was happening on shutdown.

Fixes #9794

This adds a shutdown handler to the certifier plugin so that its SSL
data can be cleaned up in a callback rather than on process shutdown.

This also reverts commenting out the debug call from SslData which was
commented out as a workaround to a crash which was happening on
shutdown.

Fixes apache#9794
@bneradt bneradt added Crash certifier certifier plugin labels Jun 7, 2023
@bneradt bneradt added this to the 10.0.0 milestone Jun 7, 2023
@bneradt bneradt requested a review from maskit June 7, 2023 22:33
@bneradt bneradt self-assigned this Jun 7, 2023
@bneradt
Copy link
Copy Markdown
Contributor Author

bneradt commented Jun 8, 2023

Rather than adding a shutdown handler, we'll address this by using the TSDbg interface which is safer in shutdown scenarios: #9809.

@bneradt bneradt closed this Jun 8, 2023
@zwoop zwoop removed this from the 10.0.0 milestone Feb 29, 2024
@bneradt bneradt deleted the fix_certifier_shutdown_crash branch June 17, 2025 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

certifier certifier plugin Crash

Projects

None yet

Development

Successfully merging this pull request may close these issues.

certifier plugin crash on master

2 participants