Bug #67013
closed
felogin setting plugin.tx_felogin_pi1.redirectMode = refererDomains causes an exception
100%
Description
Line 692 (of the current git version) uses the following regular expression:
if (preg_match('/(^|\\.)/' . $d . '$', $redirect_domain)) {
As $d in this case is a domain string, it causes an exception:
#1: PHP Warning: preg_match(): Unknown modifier 'w' in ...
The line should be changed to
if (preg_match('/(^|\\.)' . $d . '$/', $redirect_domain)) {
See attached felogin.patch.
Files
Updated by Markus Klein over 9 years ago
Can you please describe the full usecase?
Updated by Markus Klein over 9 years ago
- Status changed from New to Accepted
- Target version set to 6.2.13
Had a quick look, you're right of course.
Can you push the patch to our review system?
Updated by Markus Klein over 9 years ago
Funny, this traces back to 2009: #19728
Updated by Gerrit Code Review over 9 years ago
- Status changed from Accepted to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39600
Updated by Gerrit Code Review over 9 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39600
Updated by Gerrit Code Review over 9 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39600
Updated by Gerrit Code Review over 9 years ago
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39601
Updated by Gerrit Code Review over 9 years ago
Patch set 2 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39601
Updated by Markus Klein over 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset b2948c3af520d7122027c91840ffbc5dec1a439a.
Updated by Riccardo De Contardi about 7 years ago
- Status changed from Resolved to Closed