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

[receiver/webhookevent] Enable component #23239

Merged
merged 12 commits into from
Jun 11, 2023
Prev Previous commit
Next Next commit
check for missing server in shutdown
  • Loading branch information
shalper2 committed Jun 9, 2023
commit 889ae32a8dc07cd70b610822818a007490ba09c4
8 changes: 7 additions & 1 deletion receiver/webhookeventreceiver/receiver.go
Original file line number Diff line number Diff line change
Expand Up @@ -133,7 +133,13 @@ func (er *eventReceiver) Start(_ context.Context, host component.Host) error {

// Shutdown function manages receiver shutdown tasks. part of the receiver.Logs interface.
func (er *eventReceiver) Shutdown(_ context.Context) error {
err := er.server.Close()
var err error

// server must exist to be closed.
if er.server != nil {
err = er.server.Close()
}

shalper2 marked this conversation as resolved.
Show resolved Hide resolved
er.shutdownWG.Wait()
return err
}
Expand Down
Loading