rollbar-php-symfony-bundle icon indicating copy to clipboard operation
rollbar-php-symfony-bundle copied to clipboard

Default affected People does not work for Symfony 6 (private Services)

Open Chris53897 opened this issue 2 years ago • 7 comments

The default person_fn does not work for me. I use Symfony 6.2, version 4 of this repo. Over my PR. But i am not sure if i am doing anything wrong, or at which point is does not work.

https://docs.rollbar.com/docs/person-tracking https://docs.rollbar.com/docs/symfony#person_fn-configuration-option

The field: body.extra.token.user_identifier is set correcty in the Item-Report in Rollbar. But Rollbar does not seems to understand, that this is the user.

@danielmorell Maybe you can have a look?

Chris53897 avatar Feb 24 '23 16:02 Chris53897

Sorry for the long wait on this one @Chris53897!

Can you help me understand where you are seeing the body.extra.token.user_identifier field. Is that the raw JSON in the Rollbar platform?

I believe the raw JSON should look more like this...

{
    "body": {
        // Stuff...
    },
    "person": {
        "id": "<some id>",
        "username": "<username>",
        "email": "<email>",
    }
}

The default person function is found here https://github.com/rollbar/rollbar-php-symfony-bundle/blob/master/Factories/RollbarHandlerFactory.php for Rollbar Symfony.

danielmorell avatar Mar 17 '23 11:03 danielmorell

@danielmorell Thanks for reply. The field body.extra.token.user_identifier field was my username (as plain string) I will have a look as soon if i found some free time.

Chris53897 avatar Mar 25 '23 02:03 Chris53897

I opened this Issue on the wrong repo. Could be moved to https://github.com/rollbar/rollbar-php-symfony-bundle/issues

Chris53897 avatar May 09 '23 18:05 Chris53897

I'll close this issue, as it's in a wrong repo.

ghost avatar May 23 '23 13:05 ghost

@rollbar-bborsits ? It was already moved 4 days ago to this (correct) Repo. Please reopen

Chris53897 avatar May 23 '23 13:05 Chris53897

@Chris53897 Apology, I mixed up the repos. You're right, so I reopened it.

ghost avatar May 23 '23 13:05 ghost

thanks

Chris53897 avatar May 23 '23 13:05 Chris53897