tres/secure_channel: Fix a bug which interpolating error when logging

This commit is contained in:
Eishun Kondoh 2018-04-17 15:01:37 +09:00
parent da682cb2e4
commit 471abfceef
2 changed files with 11 additions and 4 deletions

7
CHANGELOG.md Normal file
View file

@ -0,0 +1,7 @@
# Changelog
## develop (unreleased)
### Bugs Fixed
* tres/secure_channel: Fix to interpolate terms when logging

View file

@ -580,19 +580,19 @@ defmodule Tres.SecureChannel do
end end
defp close_connection({:main_closed = disconnected_reason, reason}, state_data) do defp close_connection({:main_closed = disconnected_reason, reason}, state_data) do
warn("[#{__MODULE__}] connection terminated: Main connection down by #{reason}") warn("[#{__MODULE__}] connection terminated: Main connection down by #{inspect(reason)}")
%State{handler_pid: handler_pid} = state_data %State{handler_pid: handler_pid} = state_data
send(handler_pid, {:switch_disconnected, disconnected_reason}) send(handler_pid, {:switch_disconnected, disconnected_reason})
{:stop, :normal, %{state_data | socket: nil}} {:stop, :normal, %{state_data | socket: nil}}
end end
defp close_connection({:handler_down = _disconnected_reason, reason}, state_data) do defp close_connection({:handler_down = _disconnected_reason, reason}, state_data) do
warn("[#{__MODULE__}] connection terminated: Handler process down by #{reason}") warn("[#{__MODULE__}] connection terminated: Handler process down by #{inspect(reason)}")
{:stop, :normal, %{state_data | socket: nil}} {:stop, :normal, %{state_data | socket: nil}}
end end
defp close_connection({:trap_detected = disconnected_reason, reason}, state_data) do defp close_connection({:trap_detected = disconnected_reason, reason}, state_data) do
warn("[#{__MODULE__}] connection terminated: Trapped by #{reason}") warn("[#{__MODULE__}] connection terminated: Trapped by #{inspect(reason)}")
%State{handler_pid: handler_pid} = state_data %State{handler_pid: handler_pid} = state_data
send(handler_pid, {:switch_disconnected, disconnected_reason}) send(handler_pid, {:switch_disconnected, disconnected_reason})
{:stop, :normal, %{state_data | socket: nil}} {:stop, :normal, %{state_data | socket: nil}}
@ -606,7 +606,7 @@ defmodule Tres.SecureChannel do
end end
defp close_connection({:tcp_error, reason} = disconnected_reason, state_data) do defp close_connection({:tcp_error, reason} = disconnected_reason, state_data) do
warn("[#{__MODULE__}] connection terminated: TCP Error occured: #{reason}") warn("[#{__MODULE__}] connection terminated: TCP Error occured: #{inspect(reason)}")
%State{handler_pid: handler_pid} = state_data %State{handler_pid: handler_pid} = state_data
send(handler_pid, {:switch_disconnected, disconnected_reason}) send(handler_pid, {:switch_disconnected, disconnected_reason})
{:stop, :normal, %{state_data | socket: nil}} {:stop, :normal, %{state_data | socket: nil}}