Skip to content

Commit

Permalink
log full exception upon deserialization failure (#5121)
Browse files Browse the repository at this point in the history
  • Loading branch information
Aaronontheweb committed Jul 5, 2021
1 parent bf8a983 commit e5c0946
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/core/Akka.Remote/Endpoint.cs
Original file line number Diff line number Diff line change
Expand Up @@ -1500,7 +1500,7 @@ private bool WriteSend(EndpointManager.Send send)
{
_log.Error(
ex,
"Serializer not defined for message type [{0}]. Transient association error (association remains live)",
"Serialization failed for message [{0}]. Transient association error (association remains live)",
send.Message.GetType());
return true;
}
Expand Down Expand Up @@ -1981,8 +1981,8 @@ private void Reading()
private void LogTransientSerializationError(Message msg, Exception error)
{
var sm = msg.SerializedMessage;
_log.Warning(
"Serializer not defined for message with serializer id [{0}] and manifest [{1}]. " +
_log.Warning(error,
"Deserialization failed for message with serializer id [{0}] and manifest [{1}]. " +
"Transient association error (association remains live). {2}",
sm.SerializerId,
sm.MessageManifest.IsEmpty ? "" : sm.MessageManifest.ToStringUtf8(),
Expand Down

0 comments on commit e5c0946

Please sign in to comment.