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

Send SQS worker exceptions to defined log channel #19

Closed Daynnnnn closed 3 years ago

Daynnnnn commented 3 years ago

Using the report helper, we can push exceptions to our defined log channel before bref handles the exception.

Ideally, if a job fails, exceptions would be pushed to our desired log driver. Cloudwatch logs aren't the easiest to use, especially for people who aren't in the know on how bref is deployed. So we use flare to keep visibility on errors consistent across all of our laravel apps. Currently, the bref SQS queue handler doesn't report failed jobs to the defined log channel, this is confusing when web functions do.

Daynnnnn commented 3 years ago

Yep switching those makes sense for consistency, committed that suggestion now.

mnapoli commented 3 years ago

Thank you!