TYPO3 Core

Oops part of sentence missing:
class.external_parser.php tries to check the existence * of external parsers * with is_file().

The patch is incomplete. You need to replace all occurrences of
if (ini_get('safe_mode') ||
with
if (ini_get('safe_mode') || ini_get('open_basedir') ||
in the file class.external_parser.php

this is a anyoing problem, becase indexed_search wont work on any hosting with openbasedir settings...

sorry i've added a file by error
debug_is_tt_news.html must be removed from here

This bug is really annoying.

I completed the patch from above as adviced by Valentin.

Please check and vote for inclusion into trunk...

no news?
no one who can put this into the trunk?

Christian, did you send it to the core list for inclusion to trunk? ;)

Hi Dimitry

no sorry - didn't have the time to do this - we need several patch files (slightly different) for the different TYPO3 versions.
I'm very busy theese months... hope I can do it later.

If someone else can send the patches there - I'm also happy - of course... :-)

The solution
- if (ini_get('safe_mode') || (@is_file(...
+ if (ini_get('safe_mode') || ini_get('open_basedir') || (@is_file(...

is working good in our environment (php with fastcgi, safe_mode off, open_basedir set).

Regards Christian

ok, I sent it to the core list...

hopefully someone can adopt it to the trunk (there's a difference to 4_3+4_4)

I uploaded the three patches also here - so that others can use them - until they are integrated in the official packages...
One for TYPO3 4.2, one for 4.3 and 4.4 and one for the actual trunk....

Hoping that they are accepted soon.... :-)

/Ch

sure latest patch in core list works? I didn't test, but open_basedir is no boolean flag, but a colon separated list of accessible pathes ... probably latest patch handles this a bit naive, am I wrong?

sorry - the patch for the trunk is not testet - and you seem to be right...

maybe I use the old-fashioned way "if ini_get('open_basedir')" and if someone wants to use the new t3lib_utility_php thing then he needs to add some methods to this class...

What do you think?

The patches for 42 till 44 are working - I use them after every TYPO3 Update (sonce a few months...)

/Ch

ah - you're the author of this new utility class... :-)
please add a method to check if the option is set or not.... - I'll change the patch. Okay?

Grüessli Christian

You'll find a new patch for TYPO3 4.5 and trunk

Yes - at least 4.5.35...

just removed the @ infront of the "is_file()" calls...

[15-Jul-2014 22:58:04] http://dev.XXX.com/ - Core: Error handler (FE): PHP Warning: is_file(): open_basedir restriction in effect. File(/usr/bin/unzip) is not within the allowed path(s): (/var/www/virtual/xxx.com/:/usr/share/php/:/var/www/webapps/) in /var/www/webapps/typo3/typo3_src-4.5.35_org/typo3/sysext/indexed_search/class.external_parser.php line 175

[15-Jul-2014 22:58:04] http://dev.XXX.com/ - Core: Error handler (FE): PHP Warning: is_file(): open_basedir restriction in effect. File(/usr/bin/unrtf) is not within the allowed path(s): (/var/www/virtual/xxx.com/:/usr/share/php/:/var/www/webapps/) in /var/www/webapps/typo3/typo3_src-4.5.35_org/typo3/sysext/indexed_search/class.external_parser.php line 185

adding "|| ini_get('open_basedir') " to all theese lines still solves the problem..

eg.
instead of
if ($safeModeEnabled || @is_file($unzipPath . 'unzip' . $exe)) {
better:
if ($safeModeEnabled || ini_get('open_basedir') || @is_file($unzipPath . 'unzip' . $exe)) {

/Christian