Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description:
When the
Start
method of the named pipe is called, it will create a new named pipe and start listening for connections. If that Start method fails for any reason, some parts of the named pipe will not be initialized. This means that when theStop
method is called (as the collector does whenStart
fails), it will panic with a nil pointer dereference.This change adds a couple of nil checks to the
Stop
method to prevent this panic, because otherwise we just get the SIGSEGV signal, and we never get any error logs.Link to tracking Issue:
Testing: Running this in prod @ Cloudflare
Documentation: