-
Notifications
You must be signed in to change notification settings - Fork 62
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
SQS Exception: 404 Not Found on DeleteMessage #2
Comments
Thanks for the report! I'll work on that tomorrow and post back. |
I think I may have figured it out. In handler.php I'm using: return $app->makeWith(LaravelSqsHandler::class, [
'connection' => 'sqs',
'queue' => getenv('SQS_PREFIX') . '/' . getenv('SQS_QUEUE'),
]); |
@heathdutton I am sending you an email right now, I have more details for you about this project. Talk to you soon. |
@mnapoli Should we update documentation? |
Well, I guess that depends on the interpretation. |
👍 then maybe a note about this could help indeed. |
Otherwise environment variables will have different meanings which can lead to potential issues in the future.. |
I agree with @fabiofdsantos laravel-bridge should be using the same environment variable meanings as Laravel/Lumen. That is, the SQS_PREFIX is the URL without the queue name and SQS_QUEUE is the queue name. Then this is more in line with the idea of having one connection with multiple queues and users coming over from the Laravel/Lumen server based world (trying serverless) will have less friction (as we've all had in this thread). All that's needed is an update to the README.md I think? |
Thanks a ton for putting this repo out there. Can't wait to use it!
Using Laravel 7/latest.
I am not having any trouble inserting queued jobs, but the worker lambda function gets an error trying to use DeleteMessage. Looks like the prefix or account number is missing. I followed your handler.php script exactly. Feels like I'm missing something quite obvious.
EnvVars:
Logs:
The text was updated successfully, but these errors were encountered: