Skip to content

Update Commands to use the Fully Qualified Class Name for the 'Transaction' #1022

@lewislarsen

Description

@lewislarsen

Problem Statement

I've recently configured Traces for one of my applications and at least to me, it feels a bit strange that commands are using the command string itself as opposed to the class name.

In Laravel, you can use the class name (LetsDoSomething::class) in places like routes/console.php as opposed to something like app:lets-do-something. (This hasn't always been possible but I think it looks much neater!)

I can understand why the string itself might be more valuable but it feels a bit out of place.

Solution Brainstorm

Update https://github.com/getsentry/sentry-laravel/blob/master/src/Sentry/Laravel/Features/ConsoleSchedulingIntegration.php#L147 to send the class name to be consistent with other transactions, such as jobs.

Metadata

Metadata

Assignees

No one assigned
    No fields configured for issues without a type.

    Projects

    Status

    No status

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions