php-openid
php-openid copied to clipboard
Wishlist: Use PDO rather than Pear-DB, due to it's closer integration into the PHP core
This is something I plan to work on in the near future, however, it's more here in case anyone else wants to create the new SQL layer before I get around to it :)
I'm definitely in favor of this.
I have created a patch for this, but it's breaking on the signature validation.
https://github.com/JonTheNiceGuy/php-openid/commit/583d894d82c0781996458afe5bd7c3255b58fa57
Could someone please review what (if anything) I'm doing wrong? Thanks!
JonTheNiceGuy,
I don't think you need all that. I've done a
class Auth_OpenID_PDO_DatabaseConnection extends Auth_OpenID_DatabaseConnection
and use that with an derived
Auth_OpenID_SQLiteStore
The only thing with the Auth_OpenID_SQLiteStore is that you have to overwrite the isError() as it depends on PEAR. Working sofar but not extensively tested.
This repo is being archived. Closing issue.