brefphp / laravel-bridge

Package to use Laravel on AWS Lambda with Bref
https://bref.sh/docs/frameworks/laravel.html
MIT License
314 stars 63 forks source link

Replace AWS SDK with Async AWS #12

Closed pmayet closed 3 years ago

pmayet commented 3 years ago

This saves about 3mb

mnapoli commented 3 years ago

Thanks for the PR!

To be honest, I did not anticipate that it would change the Laravel SQS driver and force to create a separate driver. I'm afraid this changes a bit too much here (both for maintenance, familiarity for users, confidence and extra step for installation).

mnapoli commented 3 years ago

Let's put this aside for now, thanks for opening this. I think with Laravel it's a safe bet to stay with the official SDK as Laravel itself is doing the same thing, so if users use anything AWS-related provided by Laravel then the SDK will already be installed.