Как использовать __STACKTRACE__ в случае оператора или контроллеров - PullRequest
Новая
       29

Как использовать __STACKTRACE__ в случае оператора или контроллеров

5 голосов
/ 15 марта 2019

Я борюсь с предупреждением об устаревании в моем приложении Phoenix: System.stacktrace/0 outside of rescue/catch clauses is deprecated. If you want to support only Elixir v1.7+, you must access __STACKTRACE__ inside a rescue/catch. If you want to support earlier Elixir versions, move System.stacktrace/0 inside a rescue/catch.

Дело в том, что я использую Rollbax, как описано в их документации: Rollbax.report(:error, ArgumentError.exception("oops"), System.stacktrace()) и кажется странным заключать в оболочку все операторы case, которые я делаю, в блоке try / rescue. Например этот: 

case (SOME_URL |> HTTPoison.get([], [ ssl: [{:versions, [:'tlsv1.2']}] ])) do
      {:ok, %HTTPoison.Response{status_code: 200, body: body}} ->
        Poison.decode!(body, [keys: :atoms])
      {:ok, %HTTPoison.Response{status_code: 404}} ->
        Rollbax.report(:error, :not_found, System.stacktrace(), %{reason: "Not found"})
      {:ok, %HTTPoison.Response{status_code: 503}} ->
        {:error, :ehostunreach}
      {:error, %HTTPoison.Error{reason: :ehostunreach}} ->
        {:error, :ehostunreach}
      {:error, %HTTPoison.Error{reason: :timeout}} ->
        Rollbax.report(:error, :timeout, System.stacktrace(), %{reason: :timeout})
      {:error, %HTTPoison.Error{reason: reason}} ->
        Rollbax.report(:error, :unknown, System.stacktrace(), %{reason: reason})
        {:error, reason}
    end

Я не уверен, как я могу получить эти разные отчеты в одном спасении ... Как правильно написать этот отчет?

1 Ответ

2 голосов
/ 15 марта 2019

Оберните все case утверждение: 

try do
  #                           ⇓⇓⇓⇓ NOTE BANG
  case (SOME_URL |> HTTPoison.get!([], [ ssl: [{:versions, [:'tlsv1.2']}] ])) do
    %HTTPoison.Response{status_code: 200, body: body} ->
      Poison.decode!(body, [keys: :atoms])
    %HTTPoison.Response{status_code: 404} ->
      raise HTTPoison.Error, reason: :not_found
    %HTTPoison.Response{status_code: 503} ->
      {:error, :ehostunreach}
rescue
  e in [HTTPoison.Error] ->
    case e do
      %HTTPoison.Error{reason: :not_found} ->
        Rollbax.report(:error, :not_found, __STACKTRACE__, %{reason: "Not found"})
      %HTTPoison.Error{reason: :ehostunreach} ->
        {:error, :ehostunreach}
      %HTTPoison.Error{reason: :timeout} ->
        Rollbax.report(:error, :timeout, __STACKTRACE__, %{reason: :timeout})
      %HTTPoison.Error{reason: reason} ->
        Rollbax.report(:error, :unknown, __STACKTRACE__, %{reason: reason})
        {:error, reason}
    end
end
...