-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Default affected People does not work for Symfony 6 (private Services) #91
Comments
Sorry for the long wait on this one @Chris53897! Can you help me understand where you are seeing the 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 Thanks for reply. The field |
I opened this Issue on the wrong repo. Could be moved to https://github.com/rollbar/rollbar-php-symfony-bundle/issues |
I'll close this issue, as it's in a wrong repo. |
@rollbar-bborsits ? It was already moved 4 days ago to this (correct) Repo. Please reopen |
@Chris53897 Apology, I mixed up the repos. You're right, so I reopened it. |
thanks |
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?
The text was updated successfully, but these errors were encountered: