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

Provide Mysqli support #425

Closed
pebosi opened this issue Nov 20, 2008 · 5 comments
Closed

Provide Mysqli support #425

pebosi opened this issue Nov 20, 2008 · 5 comments
Assignees
Labels
Enhancement For new feature suggestions that enhance Matomo's capabilities or add a new report, new API etc.
Milestone

Comments

@pebosi
Copy link
Contributor

pebosi commented Nov 20, 2008

Hi got the following error when trying to change to mysqli using latest rev.

Fatal error: Call to undefined method mysqli::setAttribute() in /var/www/public/piwik/core/Piwik.php on line 917

regards

@mattab
Copy link
Member

mattab commented Nov 24, 2008

Attachment: DB.php hack to work with mysqli
[Db.php](http://issues.piwik.org/attachments/425/Db.php)

@mattab
Copy link
Member

mattab commented Nov 24, 2008

I received the Db.php from Emile Schenk es at donauweb dot at

Please contact him for more information on how Piwik works with MySQLi.
We would be happy to support mysqli but the provided hack is not a suitable solution. We would need a different DB class for mysqli and PDO, with a factory, etc.

Please use this ticket to submit patches. Cheers

@robocoder
Copy link
Contributor

Reprioritizing given leaks in PdoMysql.

@robocoder
Copy link
Contributor

I'm speculating that this will require a newer version of PHP without the Suhosin patch and the ZF update.

PHP 5.2.6-3ubuntu4.2 with Suhosin-Patch 0.9.6.2

ALERT - canary mismatch on efree() - heap overflow detected (attacker '127.0.0.1', file '/path/core/PluginsFunctions/Sql.php', line 44)

PHP 5.2.10 (XAMPP build)

Warning: Wrong parameter count for mysqli_stmt::bind_param() in /path/libs/Zend/Db/Statement/Mysqli.php on line 227

@robocoder
Copy link
Contributor

In [1456], fixes #425 - provide MySQLi support; added a factory method and refactored Tracker/Db.php into separate adapters (Tracker/Db/MySqli.php and Tracker/Db/PdoMySql.php).

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement For new feature suggestions that enhance Matomo's capabilities or add a new report, new API etc.
Projects
None yet
Development

No branches or pull requests

3 participants