Crash from bus.Send

[svarunan]

Crash log:

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x40 pc=0xe16308]
goroutine 14534459 [running]:
[github.com/CyCoreSystems/ari/v5/stdbus.(*bus).Send(0xc0000ebb40](http://github.com/CyCoreSystems/ari/v5/stdbus.(*bus).Send(0xc0000ebb40), {0x0, 0x0})
        /go/pkg/mod/github.com/!cy!core!systems/ari/v5@v5.3.1/stdbus/bus.go:55 +0xc8
[github.com/CyCoreSystems/ari/v5/client/native.(*Client).wsRead.func1()](http://github.com/CyCoreSystems/ari/v5/client/native.(*Client).wsRead.func1())
        /go/pkg/mod/github.com/!cy!core!systems/ari/v5@v5.3.1/client/native/client.go:393 +0x51
created by [github.com/CyCoreSystems/ari/v5/client/native.(*Client).wsRead](http://github.com/CyCoreSystems/ari/v5/client/native.(*Client).wsRead)
        /go/pkg/mod/github.com/!cy!core!systems/ari/v5@v5.3.1/client/native/client.go:378 +0xaa

Fix:
lately i keep seeing this crash happening a lot in my environment.
I've identified the crash and fix but not able to push my branch. Should I be added as a collaborator?

remote: Permission to CyCoreSystems/ari.git denied to svarunan.
fatal: unable to access 'https://github.com/CyCoreSystems/ari.git/': The requested URL returned error: 403

[Ulexus]

The usual flow on Github is to create a fork, branch from your fork, and then submit a pull request back to here from that branch.

[Ulexus]

Regardless, you are correct. That is definitely a bug. It should be calling continue there instead of trying to send a nil event to the bus.

[Ulexus]

So if you would like to submit a PR for this, I would be happy to accept it, with thanks (but it should use the current errChan implementation, not the logger).

[svarunan]

@Ulexus I'm happy to fork and raise a PR. Just some clarifying questions.

1. If we send via errChan, then by default WS will be closed, Just for DecodeEvent failure I don't want to close the whole socket connection, We can print an error message instead. What are you thoughts?

[Ulexus]

Ah, fair point. In that case, logging is fine... I was mainly concerned that we may not always have a Logger, but I just checked, and there is a discard logger by default, so that should be safe. Thanks

[svarunan]

PR is raised @Ulexus

[Ulexus]

Merged. Thanks!