Skip to content

Commit de77da2

Browse files
committed
Use args for opts
1 parent fb9c7b8 commit de77da2

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

lib/phoenix_container_example/autoscaling.ex

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ defmodule PhoenixContainerExample.Autoscaling do
1111
require Logger
1212

1313
@tab :aws_autoscaling
14-
@default_duration to_timeout(minute: 5)
14+
@default_backoff_duration to_timeout(minute: 5)
1515

1616
# Public API
1717

@@ -85,9 +85,10 @@ defmodule PhoenixContainerExample.Autoscaling do
8585
@spec rate_limit(map(), keyword()) :: {:ok, map()} | {:error, :rate_limit}
8686
defp rate_limit(event, opts) do
8787
rate_limit = opts[:rate_limit] || RateLimit
88+
backoff_duration = opts[:backoff_duration] || @default_backoff_duration
8889

8990
resource_id = Map.fetch!(event, :resource_id)
90-
{duration, event} = Map.pop(event, :backoff_duration, @default_duration)
91+
{duration, event} = Map.pop(event, :backoff_duration, backoff_duration)
9192

9293
case rate_limit.hit(resource_id, duration, 1) do
9394
{:allow, _count} ->
@@ -165,7 +166,8 @@ defmodule PhoenixContainerExample.Autoscaling do
165166

166167
defaults = [
167168
tab: @tab,
168-
rate_limit: RateLimit
169+
rate_limit: RateLimit,
170+
backoff_duration: @default_backoff_duration
169171
]
170172

171173
opts = Keyword.merge(defaults, args)

0 commit comments

Comments
 (0)