LogEvent type of UnhandledMessage was changed to INFO in Akka.TestKit (#6354). #6360
Conversation
| var warning = new Warning(rcp.Path.ToString(), rcp.GetType(), "Unhandled message from " + un.Sender + ": " + un.Message); | ||
| if(!ShouldFilter(warning)) | ||
| Print(warning); | ||
| var info = new Info(rcp.Path.ToString(), rcp.GetType(), "Unhandled message from " + un.Sender + ": " + un.Message); |
There was a problem hiding this comment.
But this is a Warning in the JVM. Are we sure this is the root cause of the issue?
There was a problem hiding this comment.
As i see in Akka.Event.DeadLetterListener message UnhandledMessage is handled as Info message instead of TestEventListener.
There was a problem hiding this comment.
There was a problem hiding this comment.
Looks like @F0b0s is right - in the real logging system unhandled messages get logged as Info unless he's not looking in the right place. I did a quick (not complete) look through the code and the DeadLetterListener is the only built-in subscriber for it.
There was a problem hiding this comment.
I found solution for #6316 and it's blocked by this PR. When it will be merged i can create MR for it.
Fixes #6354
Changes
TestEventListener produce Warning LogEvent for UnhandledMessage instead of Info. Also minor stylefixes.
Checklist
For significant changes, please ensure that the following have been completed (delete if not relevant):