-
Notifications
You must be signed in to change notification settings - Fork 394
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
Serialization of 'PDO' is not allowed - Queued Audits #879
Comments
Just to confirm, based on your stack trace you have |
Hello, it's from an upgrade. So it's the default one: From the stack trace we can see it's using the SyncQueue:
I can confirm that accesing the setting is resulting in 'sync'. ModelMy model is a bit more complex than the normal ones. I'll try to provide more informations asap regarding the model. |
Yeah, it looks like it wont serialize. The problem seems to be from the model like you suspect. If you can get us some details on what cant be serialized, we could try and test and find a way to deal with it. I haven't found a case to recreate the issue with. @Orrison - can you help? In the meantime, you may of course simply revert back to the previous version in composer.json |
@MortenDHansen sure thing. @philip1337 that makes sense. From the look of the stacktrace it seems you have an attribute that is possibly something like a QueryBuilder? More details on the attribute would help track down the issue and what could be done to fix it. |
Try disable it with #881 |
It is already disabled but since it's getting dispatched trough the SyncQueue it's still getting serialized. |
The opt-in version of v13.6.0 has been released. I hope that solves it :) |
It solved it. As mentioned before my assumption turned out correct, there was an attribute that one of the other developers Thanks for the help, I really like this library! Kind regards |
Actual Behaviour
stacktrace.txt
After upgrading to v13.6.0 with the new queued audits feature, ends up in serialization issues.
Expected Behaviour
Should save the model I guess
Steps to Reproduce
Upgrade to the new laravel-auditing version.
Possible Solutions
Downgrade a version below.
The text was updated successfully, but these errors were encountered: