-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
PdoSessionHandler not escaping identifiers #11131
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
Comments
|
Thanks for no. 4. |
What's the difference? |
I think sprintf() is used in symfony as standard. Anyway, it is not blocking. :) |
The problem is that different DBMS use different characters for quoting identifiers. And it's not recommended to use to use reserved words anyway. Neither in doctrine. So not sure if worth implementing. See https://github.com/doctrine/dbal/blob/master/lib/Doctrine/DBAL/Platforms/AbstractPlatform.php#L529 |
I will try to work on it for enhancements and send you newer version hopefully. |
I already looked into fixing it as part of #10931 |
Closing as @Tobion fixed most "fixable" problems. |
Hi,
I am using Symfony 2.3.15. I encounter following bugs when I try to implement PdoSessionHandler. I override my own class to overcome following issues. Unfortunately, the library not even throwing any exceptions for these issues.
Regards,
Asim
The text was updated successfully, but these errors were encountered: