Skip to content

Commit 4d6aa11

Browse files
Merge pull request #1070 from nextcloud/fix/readme-occ-typo
fix(docs): setting is allow_ip_ranges (plural)
2 parents a801820 + 158b519 commit 4d6aa11

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

README.md

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,15 +34,17 @@ Some other services such as office suites communicate directly with the Nextclou
3434
For Nextcloud Office and Officeonline the `wopi_allowlist` settings of the respective apps are taken into account.
3535

3636
To allow other services to bypass the terms of service check:
37-
* Set `allow_ip_ranges` to match the ip addresses of the servers in question.
3837
* Set `allow_path_prefix` to the paths that access should be granted to.
38+
* Set `allow_ip_ranges` to match the ip addresses of the servers in question.
39+
If you are using a reverse proxy, use the ip address of the application server.
40+
Access is allowed based on the x-forwarded-for header and not the source ip.
3941

40-
Default for `allow_ip_ranges` is none: ``
4142
Default for `allow_path_prefix` is none: ``
43+
Default for `allow_ip_ranges` is none: ``
4244

4345
```
44-
./occ config:app:set terms_of_service allow_ip_range --value '10.0.0.5,10.0.0.6'
4546
./occ config:app:set terms_of_service allow_path_prefix --value '/apps/onlyoffice/download'
47+
./occ config:app:set terms_of_service allow_ip_ranges --value '10.0.0.5,10.0.0.6'
4648
```
4749

4850
## 🏗️ Development setup

0 commit comments

Comments
 (0)