Allow calling with different capacities

Author: jyeshe

lib/lightning/webhook_rate_limiter.ex

@@ -37,13 +37,13 @@ defmodule Lightning.WebhookRateLimiter do
     capacity = Keyword.fetch!(opts, :capacity)
     refill = Keyword.fetch!(opts, :refill_per_second)
 
-    {:ok, %{table: :ets.new(:table, [:set]), capacity: capacity, refill: refill}}
+    {:ok, %{table: :ets.new(:table, [:set]), capacity: capacity, refill_per_second: refill}}
   end
 
-  def check_rate(bucket, cost \\ 1, name \\ __MODULE__) do
+  def check_rate(bucket, capacity \\ nil, refill \\ nil, name \\ __MODULE__) do
     name
     |> via_tuple()
-    |> GenServer.call({:check_rate, bucket, cost})
+    |> GenServer.call({:check_rate, bucket, capacity, refill})
   end
 
   def inspect_table(name \\ __MODULE__) do
@@ -53,8 +53,8 @@ defmodule Lightning.WebhookRateLimiter do
   end
 
   @impl true
-  def handle_call({:check_rate, bucket, cost}, _from, state) do
-    {:reply, do_check_rate(state, bucket, cost), state}
+  def handle_call({:check_rate, bucket, capacity, refill}, _from, state) do
+    {:reply, do_check_rate(state, bucket, capacity, refill), state}
   end
 
   @impl true
@@ -74,21 +74,19 @@ defmodule Lightning.WebhookRateLimiter do
     {:stop, :normal, state}
   end
 
-  def do_check_rate(
-        %{table: table, capacity: capacity, refill: refill_per_sec},
-        bucket,
-        cost
-      ) do
+  def do_check_rate( %{table: table} = config, bucket, capacity, refill_per_sec) do
     now = System.monotonic_time(:millisecond)
+    capacity = capacity || config[:capacity]
+    refill_per_sec = refill_per_sec || config[:refill_per_second]
 
     :ets.insert_new(table, {bucket, {capacity, now}})
     [{^bucket, {level, updated}}] = :ets.lookup(table, bucket)
 
     refilled = div(now - updated, 1_000) * refill_per_sec
     current = min(capacity, level + refilled)
 
-    if current >= cost do
-      level = current - cost
+    if current >= 1 do
+      level = current - 1
       :ets.insert(table, {bucket, {level, now}})
 
       {:allow, level}