Along the same lines as #1841896: Fix various file permissions, some more files have crept into the codebase with unnecessary executable bits (0755 perms). A patch to fix will follow. See also #1891270: [discussion] How can we prevent "invisible" regressions?.

#4 20062-before-1.png30.32 KBandymartha
#4 20062-before-2.png17.33 KBandymartha
#4 20062-after-1.png30.18 KBandymartha
#4 20062-after-2.png14.52 KBandymartha
#1 drupal-more-file-perms-2006258-1.patch280 bytesTravisCarden
PASSED: [[SimpleTest]]: [MySQL] 56,573 pass(es). View
Members fund testing for the Drupal project. Drupal Association Learn more


TravisCarden’s picture

Assigned: TravisCarden » Unassigned
Status: Active » Needs review
280 bytes
PASSED: [[SimpleTest]]: [MySQL] 56,573 pass(es). View

Status: Needs review » Needs work
Issue tags: -Quick fix, -Novice

The last submitted patch, drupal-more-file-perms-2006258-1.patch, failed testing.

TravisCarden’s picture

Status: Needs work » Needs review
Issue tags: +Quick fix, +Novice
andymartha’s picture

Status: Needs review » Reviewed & tested by the community
14.52 KB
30.18 KB
17.33 KB
30.32 KB

I can confirm that after applying patch drupal-more-file-perms-2006258-1.patch to a fresh installation of Drupal 8.x-dev by TravisCarden in #1, the files mentioned change from being executable to read and write only. See screenshots.





alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 8f85c84 and pushed to 8.x. Thanks!

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