dusterio / laravel-plain-sqs

Custom SQS connector for Laravel (or Lumen) that supports third-party, plain JSON messages
MIT License
131 stars 92 forks source link

Queue name is not passed correctly when checking the job #30

Open shawnm0705 opened 5 years ago

shawnm0705 commented 5 years ago

Issue: Queue names in sqs-plain.php is not detected. Always use 'default-handler' instead.

Reason of the issue: in the latest Laravel (v5.8), in Illuminate/Queue/Queue.php it is createPayload($job, $queue, $data = '') (ref: https://laravel.com/api/5.8/Illuminate/Queue/Queue.html#method_createPayload)

However, in the /src/Sqs/Queue.php here, it is createPayload($job, $data = '', $queue = null), which is not align to the parent. So that the queue name is passed as $data here.