Skip to content
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

Use PDO and parameterised queries #16094

Closed
Simon-Davies opened this issue May 17, 2017 · 4 comments
Closed

Use PDO and parameterised queries #16094

Simon-Davies opened this issue May 17, 2017 · 4 comments

Comments

@Simon-Davies
Copy link

Simon-Davies commented May 17, 2017

In the light of the recent SQL Injection vulnerability in Joomla 3.7 would it not be a good idea to rewrite Joomla's code to use PDO and parameterised queries?

I know this would be a lot of work but I'm willing to help.

@mbabker
Copy link
Contributor

mbabker commented May 17, 2017

It's not possible during 3.x, ext/mysql doesn't support prepared statements and it's not worth it to emulate this, especially as we're dropping it in 4.0.

As for the rest of the package, the bulk of the work is already done but the implementations are more reliant on each driver's configuration so right now I'm not sure even with all the work ported over we could write SQL and handle parameterized queries all that well without having an extra layer of string processing in our query processing code.

This is the stuff that needs to be ported over:

@SniperSister
Copy link
Contributor

Count me in here too, would love to see that happen!

@mbabker
Copy link
Contributor

mbabker commented May 31, 2017

#16402 would make use of the Framework database package in 4.0. Includes making prepared statement support available and PDO PostgreSQL connections.

@brianteeman
Copy link
Contributor

closed as this work is in progress as mentioned above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants