TYPO3 Core

Now in function getOpenIDConsumer FileStore of openID-lib is used. There is also a method SQLStore.php, so it should be no problem to switch.

just make sure it's using the TYPO3 DB API

Work is in progress.

The patch is attached. This version is tested with trunk revision #6513 and the following OpenID providers and servers:
- Yahoo
- Google
- SimpleID

Since some users already use OpenID or backported it to 4.2, they may have troubles logging in to the system because of the store change (missing database tables). Therefore this patch includes elementary workaround that creates database on the fly if they are not there. This can be removed in TYPO3 4.4 but I suggest to release 4.3 with this code.

I was trying out the patch without luck. DB tables are in place.

I must admit OpenID never worked for me here.

I'm using Verisign as OpenID provider. I'll try to debug it.

Dmitry, thank you for the patch.

I also had no luck, it did not work with BE login. But OpenID worked for me before applying the patch.
I got the same effect as described in #21647 : OpenID provider redirects me to http://lenny.test/typo3/sysext/openid/ instead of the backend.

Also, I don't like the constructor checking and creating missing database tables. It's separate one-time code which blows up the class too much. install tool should do this. This is TYPO3 best practice and users are used to go through the database compare stuff for any other DB changes. It does not hurt. We had that before in trunk. IMHO this is a major issue I see with your patch.

I found also one minor issue:
DB compare in the install tool always shows:

ALTER TABLE tx_openid_nonce_store CHANGE salt salt char(40);
Current value: char(40) default ''

Pressing the "write to database" button does not change anything.

P.S. Tested by using launchpad.net as OpenID provider.

Marcus, what would you say if somebody reported a problem to you like this: "I was trying out the patch without luck"? This is impossible to fix because it contains zero of information about the problem you have.

Steffen, I cannot confirm those redirections. I afraid you have to search your workspace for issues. Regarding table creation: this was for those who already use OpenID. The idea was to be user friendly and let them upgrade the system automatically. I will get rid of user friendliness in the new version of the patch. This would be true TYPO3 way: make it harder for users ;)

DB upgrate:
Dmitry, you know TYPO3 core is already very ugly code design. Upgrading database routines should be kept at a central place, since it is a repeating routine. With your solution we would later have duplicate code in the core, e.g. in each sysext.
On the other hand using the install tool for DB upgrade is well known and gives you full control about what happens to database tables.
You are right, this makes it harder for users to upgrade, but in a way, they are used with the above mentioned advantages.

Redirections:
I have already checked devlog/syslog for errors, but unfortunately there are none. Please give me some more time to debug it.

rev 6580