Instrument laravel queues

asgrim commented 4 years ago

Fixes #35

[x] Detect if we're running sync/async
- sync does not need to call send() since SendRequestToScout middleware will
- async will need to call send() after each job is processed (since a single worker is a single long-running thread)
[x] Figure a clean way to "reset" the request after each job is processed (in "async" mode)
[x] Find a way to fake runningInConsole() on older Laravel versions
[x] Pin to a new release of scout-apm-php
[x] Changelog update (1.2.0 since new feature)

asgrim commented 4 years ago

Note; in real terms, this likely requires core agent 1.2.7 as typically, DB instrumentation happens before jobs are executed; therefore the fixes done in 1.2.7 are required. Upstream PHP update will fix this though, as we don't pin the core agent version in this library.

asgrim commented 4 years ago

It has; I'll make a new scout-apm-php release though and pin this PR to that to ensure we pull the latest fixes :+1:

asgrim commented 4 years ago

Feature is added in 1.2.0

scoutapp / scout-apm-laravel

Instrument laravel queues #36