#2540416: Move update.php back to a front controller re-introduced update.php - but with 755 file permissions.

Unless I'm missing something, this should be 644.

CommentFileSizeAuthor
2560603-update.php-file-permissions.patch206 bytesswentel
#4 2560603-02.file-permissions.patch206 bytesksenzee
#2 2560603-update.php-file-permissions.patch69 bytesswentel
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

swentel created an issue. See original summary.

swentel’s picture

swentel CreditAttribution: swentel commented
Issue summary: View changes
Status: Active » Needs review
FileSize
2560603-update.php-file-permissions.patch69 bytes
heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC commented
Status: Needs review » Reviewed & tested by the community

Yup, it changes the file permissions.

ksenzee’s picture

ksenzee
she/her
Location Seattle area
CreditAttribution: ksenzee at Smartsheet commented
Status: Reviewed & tested by the community » Needs review
FileSize
2560603-02.file-permissions.patch206 bytes

Looks like core/lib/Drupal/Core/Update/UpdateKernel.php also got an unintended permissions change from that issue.

swentel’s picture

swentel CreditAttribution: swentel commented
Status: Needs review » Reviewed & tested by the community

Yep, good catch.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Status: Reviewed & tested by the community » Fixed

Committed aec0210 and pushed to 8.0.x. Thanks!

  • alexpott committed aec0210 on 8.0.x 
    Issue #2560603 by swentel, ksenzee: update.php has 755 file permission

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.