Skip to content

Commit d0fa596

Browse files
committed
fix: move regex out of the module scope
1 parent 2f0e737 commit d0fa596

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

lib/sequin/consumers/sqs_sink.ex

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,6 @@ defmodule Sequin.Consumers.SqsSink do
1919
field :is_fifo, :boolean, default: false
2020
end
2121

22-
@sqs_url_regex ~r/^https:\/\/sqs\.(?<region>[a-z0-9-]+)\.amazonaws\.com\/\d{12}\/[a-zA-Z0-9_-]+(?:\.fifo)?$/
23-
2422
def changeset(struct, params) do
2523
struct
2624
|> cast(params, [:queue_url, :region, :access_key_id, :secret_access_key, :is_fifo])
@@ -87,7 +85,8 @@ defmodule Sequin.Consumers.SqsSink do
8785
8886
"""
8987
def region_from_url(queue_url) do
90-
case Regex.named_captures(@sqs_url_regex, queue_url) do
88+
sqs_url_regex = ~r/^https:\/\/sqs\.(?<region>[a-z0-9-]+)\.amazonaws\.com\/\d{12}\/[a-zA-Z0-9_-]+(?:\.fifo)?$/
89+
case Regex.named_captures(sqs_url_regex, queue_url) do
9190
%{"region" => region} -> region
9291
_ -> {:error, Sequin.Error.validation(summary: "Invalid SQS queue URL format")}
9392
end

0 commit comments

Comments
 (0)