asynkron / protoactor-go

Proto Actor - Ultra fast distributed actors for Go, C# and Java/Kotlin
http://proto.actor
Apache License 2.0
5.02k stars 521 forks source link

feat: publish should only accept proto.Message #1010

Closed qazwsxedckll closed 8 months ago

qazwsxedckll commented 8 months ago

If topic actor is on another node, only proto.Message is accepted. Otherwise endpointSupervisor will crash and the node will be blocked.

qazwsxedckll commented 8 months ago

@rogeralsing

I have reproduce the problem here.

Just cd to the folder and run go test -v -tags=integraion ..

The node seems never become healthy again which I cannot figure out why and I am worried about unhealthy node existing without anyone knowing it.