jmurphyweb

jmurphyweb

How to gracefully timeout a web request

Hi all,

I have an intermittent bug in production which I am trying to track down which causes requests to take a really long time:

Heroku times out all requests after 30 seconds, meaning the response never get’s sent for these slow requests and I never get alerted of the error.

Ideally I would force my own 30 second timeout to kill the request early. I’ve looked into cowboy’s idle_timeout option but it seems to have strange behaviour, not returning an error message to the user but rather a chrome error page and the client automatically attempts a refresh.

I am thinking of setting up a genserver that does a Process.send_after(__MODULE__, {:kill_if_open, request_pid}, 30_000)
and run Process.exit(pid, :kill) in the :kill_if_open handler.

Does anyone have any feedback on this approach? If it is bad practice, I could simply log/report that request and then investigate later.

First Post!

jmurphyweb

jmurphyweb

I’ve now implemented it:

defmodule Myapp.RequestTimeout do
  use GenServer

  require Logger

  @timeout :timer.seconds(2)

  def start_link(_) do
    GenServer.start_link(__MODULE__, %{}, name: __MODULE__)
  end

  @impl true
  def init(_) do
    {:ok, nil}
  end

  @impl true
  def handle_cast({:register_request, log_meta, request_pid}, state) do
    Process.send_after(__MODULE__, {:kill_if_open, log_meta, request_pid}, @timeout)
    {:noreply, state}
  end

  @impl true
  def handle_info({:kill_if_open, log_meta, request_pid}, state) do
    Logger.metadata(log_meta)

    Logger.error("RequestTimeoutError: #{Logger.metadata()[:request_id]}")

    Process.exit(request_pid, :timeout)

    {:noreply, state}
  end

  #
  # API
  #

  def register_request(log_meta) do
    GenServer.cast(__MODULE__, {:register_request, log_meta, self()})
  end
end

defmodule Myapp.RequestTimeout.Plug do
  def init(opts) do
    opts
  end

  def call(conn, _) do
    IO.inspect(Logger.metadata()[:request_id], label: "request_id")

    Myapp.RequestTimeout.register_request(Logger.metadata())

    conn
  end
end

But there is no response sent to the user, the browser just hangs.

If anyone has any feedback or suggestions I’d be grateful to hear.

Thanks

Where Next?

Popular in Questions Top

minhajuddin
I have seen a lot of code which picks the first element from a list using Enum.at(0) instead of List.first. Is there a reason why people ...
New
New
9mm
I am constructing a JSON object (map) and I need to conditionally set a field. I’m trying to write proper elixir-way code… and I’m at a l...
New
aalberti333
As the title describes, I’m trying to run Enum.map() over a list of key/value pairs, where the value is a map. My data looks like this: ...
New
siddhant3030
Hi, I have to write a raw query for one of my project. But till now I have used ecto queries and don’t have much experience writing raw ...
New
marius95
Hello everyone, I try to use an Javascript Event Handler in my root.html.leex file. Therefore I created a function in the app.js file: ...
New
Harrisonl
We have an ECS cluster with 4 services, where each task joins a single cluster, via discovery ECS discovery service. Currently when I de...
New

Other popular topics Top

joaquinalcerro
Hi there, I am working with Ecto-Postgresql and I need to call all of the records from a specific table but the table has 40,000 records...
New
JakeBecker
TL;DR: I’ve just released an implementation of Microsoft’s IDE-independent Language Server Protocol for Elixir. It adds language support ...
1144 55125 245
New
stefanchrobot
What’s the safe way to decode a JSON string into a struct? I want to avoid calling String.to_atom. Jason.decode can give me a map with st...
New
chrismccord
Phoenix 1.4.0 released Phoenix 1.4 is out! This release ships with exciting new features, most notably with HTTP2 support, improved deve...
688 31586 112
New
bsollish-terakeet
Credo is smart enough to check for (something like) this: assert length(the_list) == 0 with this response: Checking if an enum is empt...
New
shijith.k
I am trying to start a new phoenix project with elixir 1.9, but mix phx.new does not work. It says that ** (Mix) The task "phx.new" could...
New

We're in Beta

About us Mission Statement