[TYPO3-core] RFC #12706: Bug: OpenID redirects to wrong page and does not create a BE session

[Steffen Müller]

Hi,

this is a SVN patch request.

Type: Bugfix, no-brainer

Bugtracker references: http://bugs.typo3.org/view.php?id=12706

Branches: Trunk

Problem:
OpenID login does not work. Login triggers the following warning:
PHP Warning:
file_put_contents(/path/to/site/typo3temp:tx_openid/.htaccess): failed
to open stream: No such file or directory in
/path/to/typo3_src/typo3_src-trunk/typo3/sysext/openid/sv1/class.tx_openid_sv1.php
line 330

The openid service class uses wrong constant PATH_SEPARATOR:

Additional information:
Line 325 of openid/sv1/class.tx_openid_sv1.php
$openIDStorePath = PATH_site . 'typo3temp' . PATH_SEPARATOR . 'tx_openid';

This results in .../typo3temp:tx_openid
But it should be .../typo3temp/tx_openid

This problem exists since rev.6451

Solution:
DIRECTORY_SEPARATOR should be used instead of PATH_SEPARATOR

$openIDStorePath = PATH_site . 'typo3temp' . DIRECTORY_SEPARATOR .
'tx_openid';
This correctly results in .../typo3temp/tx_openid

This one should make it into 4.3-RC2.

-- 
cheers,
Steffen

TYPO3 Blog: http://www.t3node.com/
Blubber on Twitter: http://twitter.com/t3node



-------------- next part --------------
A non-text attachment was scrubbed...
Name: bug_12706.diff
Type: application/pgp-keys
Size: 767 bytes
Desc: not available
URL: <http://lists.typo3.org/pipermail/typo3-team-core/attachments/20091124/deb6d54c/attachment.key>