Skip to content

Commit ded57bd

Browse files
committed
Some more minor changes to the README.
1 parent 00ce2c8 commit ded57bd

File tree

1 file changed

+18
-8
lines changed

1 file changed

+18
-8
lines changed

Diff for: README.md

+18-8
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,12 @@ I believe these constraints can be demonstrated with only two groups:
2424
the admins (us), and customer's developers. If I've made a mistake, it
2525
will have to be amended with an additional customer (and group).
2626

27+
# The Goal
28+
29+
We should be able to run the scripts in pg-test and have them work,
30+
considering the previous few paragraphs. Can you modify
31+
pg-test/02-create-permissions so that the rest of the tests pass?
32+
2733
# Running the Tests
2834

2935
These two tests will create system and postgres users on your
@@ -38,7 +44,7 @@ To run the tests, simply 'cd' into the directory, and run the scripts
3844
in order. On my machine, this is how the filesystem tests (POSIX ACL
3945
support required) are run:
4046

41-
```Bash
47+
```
4248
sudo ./01-create-actors.sh
4349
sudo ./02-create-acls.sh
4450
sudo ./03-run-tests.sh
@@ -49,7 +55,7 @@ sudo ./05-destroy-actors.sh
4955
There's no output, which means that the test succeeded. The postgres
5056
test, on the other hand, fails at step #4:
5157

52-
```Bash
58+
```
5359
sudo ./01-create-actors.sh
5460
sudo ./02-create-permissions.sh
5561
sudo ./03-run-tests.sh
@@ -125,21 +131,22 @@ thing. When you create a new directory for the customer,
125131

126132
1. Grant read/write permissions to the customer-devs group:
127133

128-
setfacl -m group:customer-devs:rwx <dir>
134+
`setfacl -m group:customer-devs:rwx <dir>`
129135

130136
2. Grant read-only permissions to the anonymous user:
131137

132-
setfacl -m user:anonymous:rx <dir>
138+
`setfacl -m user:anonymous:rx <dir>`
133139

134140
3. Set customer-devs defaults for newly-created files:
135141

136-
setfacl -d -m group:customer-devs:rwx <dir>
142+
`setfacl -d -m group:customer-devs:rwx <dir>`
137143

138144
4. Set anonymous defaults for newly-created-files:
139145

140-
setfacl -d -m user:anonymous:rx <dir>
146+
`setfacl -d -m user:anonymous:rx <dir>`
141147

142-
If the directory is non-empty here, find/xargs can be used.
148+
If the directory is non-empty here, find/xargs can be used. This is
149+
what the filesystem test does, and it works.
143150

144151
# Database Examples
145152

@@ -161,10 +168,13 @@ permissions to the (server-level) role.
161168
## Postgres
162169

163170
Postgres has no (obvious?) way to achieve this. The closest I was able
164-
to come can be found in the pg-tests/02-create-permissions.sh file. It
171+
to come can be found in the pg-test/02-create-permissions.sh file. It
165172
is not pretty; and doesn't fully work besides. When a new user is
166173
created in 04-add-new-user-and-retest.sh, some manual work is required
167174
to grant him the correct permissions.
168175

169176
If there are 100 databases on the server already, that could be a lot
170177
of error-prone work.
178+
179+
Is there a way to modify 02-create-permissions.sh so that
180+
04-add-new-user-and-retest.sh will work?

0 commit comments

Comments
 (0)