Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.See patch for D7 from the parent issue.
#323110-52: Remove the open_basedir requirement check
Remaining todo:
Review?
commit patch
Comments
Comment #2
joseph.olstadPatch:
https://www.drupal.org/files/issues/323110_52_remove_open_basedir_requir...
credit to:
git commit -m 'Issue #2926150 by nenne, gpk, BarisW, MrHaroldA, webflo: Remove the open_basedir requirement check'Comment #3
joseph.olstadComment #4
joseph.olstadComment #5
joseph.olstadnot sure if the patch is working yet or not
Comment #6
joseph.olstadI tested the above patch, it works.
Comment #7
David_Rothstein CreditAttribution: David_Rothstein as a volunteer commentedThis can probably go in, but the Drupal 8 issue needs to get fixed first - postponing it on that.
Comment #8
MustangGB CreditAttribution: MustangGB commentedRemoving tag until D8 fix goes in, instead bumping priority so it doesn't get forgotten.
Comment #9
MustangGB CreditAttribution: MustangGB commentedOn second thoughts, this probably makes more sense.
Comment #10
joseph.olstadThis just was pushed into 8.8.x , the 8.x issue is 'fixed'
Setting this issue to RTBC, ready to go it in !
Comment #11
joseph.olstadComment #12
joseph.olstadThis change was made in D8.
Patch:
https://www.drupal.org/files/issues/323110_52_remove_open_basedir_requir...
credit to:
git commit -m 'Issue #2926150 by nenne, gpk, BarisW, MrHaroldA, webflo: Remove the open_basedir requirement check'Comment #13
MustangGB CreditAttribution: MustangGB commentedComment #14
MustangGB CreditAttribution: MustangGB commentedComment #15
poker10 CreditAttribution: poker10 at ActivIT s.r.o. commentedAdding a patch from the parent issue (#323110-53: Remove the open_basedir requirement check ) here so we can run tests properly.
I have tested it manually and simpletest still works when this check is removed and open_basedir is enabled.
Comment #16
poker10 CreditAttribution: poker10 at ActivIT s.r.o. commentedAnd the patch :)
Comment #17
mcdruidJust to spell out the tl;dr of the rather long and meandering parent issue:
PHP's open_basedir used to cause problems with simpletest's usage of curl, so a requirements check was added pointing out the conflict with the testing framework.
In #1671200: Simpletest broken: CURLOPT_COOKIEJAR cannot be NULL (and maybe others) the conflicts with open_basedir were resolved.
So the requirements check is no longer required, and in fact could be harmful as it recommends disabling a security feature unnecessarily.
The patch looks good, thanks!
Comment #21
poker10 CreditAttribution: poker10 at ActivIT s.r.o. commentedThanks everyone!
Added credits from the parent issue.