Skip to content

Commit

Permalink
Fix mailbox post user message type check on message envelope asynkron…
Browse files Browse the repository at this point in the history
  • Loading branch information
Kunduin committed Jan 21, 2023
1 parent 54ea902 commit 8f7b39c
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 1 deletion.
10 changes: 10 additions & 0 deletions actor/mailbox.go
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,7 @@ func (m *defaultMailbox) PostUserMessage(message interface{}) {
}

// is it an envelope batch message?
// FIXME: check if this is still needed, maybe MessageEnvelope can only exist as a pointer
if env, ok := message.(MessageEnvelope); ok {
if batch, ok := env.Message.(MessageBatch); ok {
messages := batch.GetMessages()
Expand All @@ -72,6 +73,15 @@ func (m *defaultMailbox) PostUserMessage(message interface{}) {
}
}
}
if env, ok := message.(*MessageEnvelope); ok {
if batch, ok := env.Message.(MessageBatch); ok {
messages := batch.GetMessages()

for _, msg := range messages {
m.PostUserMessage(msg)
}
}
}

// normal messages
for _, ms := range m.middlewares {
Expand Down
2 changes: 1 addition & 1 deletion actor/process_registry.go
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ package actor
import (
"sync/atomic"

murmur32 "github.com/spaolacci/murmur3"
murmur32 "github.com/twmb/murmur3"

cmap "github.com/orcaman/concurrent-map"
)
Expand Down

0 comments on commit 8f7b39c

Please sign in to comment.