Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add opts.queue_url to the error message trying to receive messages #79

Closed
rogerweb opened this issue Oct 6, 2023 · 0 comments
Closed

Comments

@rogerweb
Copy link
Contributor

rogerweb commented Oct 6, 2023

My application consumes messages from multiple SQS queues. If a queue is deleted, I get an error message like this:

[error] Unable to fetch events from AWS. Reason: {:http_error, 400, %{
  code: "AWS.SimpleQueueService.NonExistentQueue",
  message: "The specified queue does not exist for this wsdl version.",
  type: "Sender",
  request_id: "5436c6c6-1c58-5e85-bf11-9b5c27e3bc93",
  detail: nil
}}

However, I can't tell which queue the error refers to because AWS doesn't include its name in the response neither broadway_sqs includes it in the error message:

  defp wrap_received_messages({:error, reason}, _) do
    Logger.error("Unable to fetch events from AWS. Reason: #{inspect(reason)}")
    []
  end

Would it be possible to include the opts.queue_url in the error message?

rogerweb added a commit to rogerweb/broadway_sqs that referenced this issue Oct 6, 2023
rogerweb added a commit to rogerweb/broadway_sqs that referenced this issue Oct 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant