Skip to content

Commit 1066373

Browse files
authored
Merge pull request #4 from ItsFrankieD/patch-1
Update README.md
2 parents 3588e00 + 3092796 commit 1066373

File tree

1 file changed

+13
-13
lines changed

1 file changed

+13
-13
lines changed

README.md

+13-13
Original file line numberDiff line numberDiff line change
@@ -3,21 +3,21 @@
33
This script sets the permissions and ownership of the files of a Drupal
44
installation.
55

6-
This is loosely based on the information provided by Drupal documentation page
6+
This is loosely based on the information provided by the Drupal documentation page
77
"[Securing file permissions and ownership](https://www.drupal.org/node/244924)".
88

99
## Details
1010

1111
For security reasons, the code files of a website should not be writable. At the
1212
same time, the website should be able to create files (for example, when a user
13-
uploads an image). This means that there two types of files and folders: content
13+
uploads an image). This means that there are two types of files and folders: content
1414
and code.
1515

16-
There will be two users involved: a regular UNIX user, we'll call they the
16+
There will be two users involved: a regular UNIX user, we'll call them the
1717
deploy user, that is in charge of managing the code (typically deploying new
1818
releases), and the user under which the web server process is running.
1919

20-
This scripts tries to secure the site using the following scheme:
20+
This script tries to secure the site using the following scheme:
2121

2222
- Code is owned by the deploy user and by the web server's
2323
group. Deploy user can write, web server group only read.
@@ -40,9 +40,9 @@ In UNIX terms:
4040

4141
## Installation
4242

43-
Clone or donwload the repository content to your server.
43+
Clone or download the repository content to your server.
4444

45-
Link to `drupal_fix_permissions.sh` in the `/usr/local/bin` or another folder present in users' PATH.
45+
Link to `drupal_fix_permissions.sh` in the `/usr/local/bin` or another folder present in the user's PATH.
4646

4747
If you are using `autofix-drupal-perms.sh`, link it as well. Because it expects `drupal_fix_permissions.sh` to be at `/usr/local/bin` make sure that path exists or edit the autofix script.
4848

@@ -71,10 +71,10 @@ folder and using `deploy` as the deploy user.
7171

7272
## Strategy
7373

74-
The scripts checks if the target folder is a Drupal installation and stops if
74+
The script checks if the target folder is a Drupal installation and stops if
7575
it is not detected.
7676

77-
Once checked, it fixes the ownership of all folder and files (because it is the
77+
Once checked, it fixes the ownership of all folders and files (because it is the
7878
same for content and code). Then, it fixes the code and later the content.
7979

8080
The script assumes that `files` and `private` folders under `sites` are content
@@ -91,8 +91,8 @@ folder and fixes permissions accordingly: it fixes ownership (owner: deploy
9191
user, group: web server) and removes any permissions for other users.
9292

9393
It doesn't apply standard permissions of code files because in `vendor` folders
94-
there are some files that needs to be executable. It would be hard to detect all
95-
the cases that needs executable permissions so the script doesn't handle
94+
some files need to be executable. It would be hard to detect all
95+
the cases that need executable permissions so the script doesn't handle
9696
permissions for the owner or the group and just removes all permissions for
9797
other users.
9898

@@ -103,15 +103,15 @@ run again to remove all permissions on other users.
103103

104104
## Performance
105105

106-
The script only changes the files and folder with the wrong permissions or
106+
The script only changes the files and folders with the wrong permissions or
107107
ownership, making it very fast when only a few files or folders need a fix. For
108108
really big installations this is very important as other scripts apply the
109109
permissions and ownership regardless are needed o not.
110110

111111
## Root permissions
112112

113-
Giving root permissions to regular user is dangerous. Luckily, there's a simple
114-
script, `autofix-drupal-perms.sh`, to allow regular users fix their sites
113+
Giving root permissions to regular users is dangerous. Luckily, there's a simple
114+
script, `autofix-drupal-perms.sh`, to allow regular users to fix their sites
115115
without risking the security.
116116

117117
This script has no parameters, so it can be easily added to the sudoers. When

0 commit comments

Comments
 (0)