Problem
If admin specified incorrect path to file in $conf['lock_inc'] in settings.php then bootstrap fails.

Proposed solution
Provide a fallback. We should add a watchdog message that specified file path is incorrect and use lock system from core.

CommentFileSizeAuthor
drupal-provide-a-fallback-for-incorrect-path-to-lock-inc-file.patch1013 bytesspleshka

Comments

podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
commented
Issue tags: +Needs issue summary update

This is follow-up from a few hours seen issue, so better do create link for understanding
Patch looks good, can we cover it by test?

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Needs work
Issue tags: +Needs tests

if D8 affected then move the issue to 8.x first

David_Rothstein’s picture

David_Rothstein commented

I don't understand, why isn't the require_once error which PHP throws good enough here?

If someone puts an incorrect value of this variable in settings.php, they'll already get an error somewhere (logs or on screen depending on the server's configuration) when they try to access the site. Just like happens with any other PHP fatal error.

Status: Needs work » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.