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

Configuring SQS with a Queue name in PascalCase appears to try and use the lowercase version #973

Closed
zanehanasoft opened this issue Jul 23, 2024 · 2 comments

Comments

@zanehanasoft
Copy link

zanehanasoft commented Jul 23, 2024

Describe the bug
When I setup an SQS queue in Wolverine, if I have a queue like EmailManagerLog2 it will try to look for emailmanagerlog2 instead. Because SQS queue names are case sensitive we have having issues linking our code to our SQS queue

To Reproduce
Steps to reproduce the behavior:

  1. Have an existing queue in SQS called EmailManagerLog2
  2. Use .NET Core 8 ASP.NET Core
  3. WolverineFx nuget package 2.6 (I've also tested on the latest version 2.15.2)
  4. Setup Wolverine as follows.
    Configuration
  5. After debugging the options after the configuration is complete is appears to create 2 queue configurations.
    AmazonSQSObjectInfo
    SQSQueueInfo
    SQSQueueInfo2
    And after the app tries to run I see this Exception.
    Exception

If I create the queue as a lowercase version it works but the actual queue I want is PascalCase and I wont be changing it because it is consumed by another service.

Expected behavior
Should not lower the case of the Queue.

UPDATE: I've pulled the source code directly and have found where the issue is. I don't have permission to create a PR though so I've attached this patch.

0001-add-a-fix-to-support-finding-an-AmazonSQSQueue-objec.patch

@jeremydmiller
Copy link
Member

@zanehanasoft Got it, and applying it now. Why didn't you have permission to do a PR though?

@zanehanasoft
Copy link
Author

oh sorry I meant I didn't have the ability to push my branch, when I tried I get an error saying I don't have write access

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

2 participants