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
F7125693: D1012.diff
Mon, Apr 21, 02:59
F7121173: D1012.id2582.diff
Mon, Apr 21, 01:26
F7097001: D1012.id2582.diff
Sun, Apr 20, 16:01
Unknown Object (File)
Sat, Apr 19, 14:51
Unknown Object (File)
Thu, Apr 17, 17:13
Unknown Object (File)
Wed, Apr 16, 20:07
Unknown Object (File)
Wed, Apr 16, 15:04
Unknown Object (File)
Wed, Apr 16, 07:40
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.