Page MenuHomeDevCentral

Set chmod to 755 instead of 711 for web static directories
ClosedPublic

Authored by dereckson on Sep 7 2017, 08:00.
Tags
None
Referenced Files
F3746962: D1012.id2582.diff
Fri, Nov 15, 23:56
F3746905: D1012.id2583.diff
Fri, Nov 15, 23:42
F3746463: D1012.diff
Fri, Nov 15, 21:27
Unknown Object (File)
Sat, Nov 9, 17:07
Unknown Object (File)
Tue, Nov 5, 05:26
Unknown Object (File)
Sun, Nov 3, 18:42
Unknown Object (File)
Sun, Nov 3, 00:55
Unknown Object (File)
Sat, Nov 2, 21:09
Subscribers
None

Details

Summary

A Forbidden was served, as nginx wasn't allowed to read
the content of directory, and so know if there is or not
an index.html page.

Test Plan

https://assets.dereckson.be/fonts should be world readable

Diff Detail

Repository
rOPS Nasqueron Operations
Lint
Lint Passed
Unit
No Test Coverage
Branch
chmod755 (branched from master)
Build Status
Buildable 1586
Build 1834: arc lint + arc unit

Event Timeline

Works fine, and is equipotent (no modification on second run).

Ysul
$ salt-call --local state.apply roles/legacy-webserver/be/dereckson
local:
----------
          ID: /var/wwwroot/dereckson.be/assets
    Function: file.recurse
      Result: True
     Comment: Recursively updated /var/wwwroot/dereckson.be/assets
     Started: 08:02:23.208925
    Duration: 2772.347 ms
     Changes:   
              ----------
              /var/wwwroot/dereckson.be/assets/.:
                  ----------
                  mode:
                      0755

Summary for local
------------
Succeeded: 1 (changed=1)
Failed:    0
------------
Total states run:     1
Total run time:   2.772 s
$ salt-call --local state.apply roles/legacy-webserver/be/dereckson
local:
----------
          ID: /var/wwwroot/dereckson.be/assets
    Function: file.recurse
      Result: True
     Comment: The directory /var/wwwroot/dereckson.be/assets is in the correct state
     Started: 08:02:48.127904
    Duration: 788.4 ms
     Changes:   

Summary for local
------------
Succeeded: 1
Failed:    0
------------
Total states run:     1
Total run time: 788.400 ms
This revision is now accepted and ready to land.Sep 7 2017, 08:04
This revision was automatically updated to reflect the committed changes.