Skip to content

Commit 9f9ffb9

Browse files
committed
docs(README): address review feedback for Apache reverse proxy section
Signed-off-by: Oleksander Piskun <oleksandr2088@icloud.com>
1 parent 343560f commit 9f9ffb9

1 file changed

Lines changed: 4 additions & 5 deletions

File tree

README.md

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ docker run \
7777

7878
##### On the Docker Host
7979
##### Creation of Cert folder (if necessary)
80-
`mkdir -p /some/path/{certs,}`
80+
`mkdir -p /some/path/certs`
8181
##### Open ports (based on Almalinux - RHEL Distros)
8282
```
8383
firewall-cmd --permanent --zone=public --add-port=8780/tcp
@@ -99,8 +99,6 @@ docker run \
9999
-d ghcr.io/nextcloud/nextcloud-appapi-harp:release
100100
```
101101

102-
> **Warning:** Do not forget to change the **HP_SHARED_KEY** value to a secure one!
103-
104102
> **Note:** You have to configure the **NC_INSTANCE_URL** value with your public Nextcloud url and the **HP_TRUSTED_PROXY_IPS** value with your local network (CDIR) that hosts your reverse proxy and your Nextcloud instance.
105103
106104
---
@@ -173,6 +171,7 @@ Add the following lines (before the existing configuration)
173171
# AppAPI Configuration
174172
ProxyPass /exapps/ http://<IP_host2_docker>:8780/exapps/
175173
ProxyPassReverse /exapps/ http://<IP_host2_docker>:8780/exapps/
174+
ProxyTimeout 1800
176175
```
177176

178177

@@ -195,7 +194,7 @@ Based on a infrastructure With 3 hosts :
195194
### On the Nextcloud Web Interface - Daemon Register
196195
Add the following configuration :
197196
```
198-
Daemon Configuraiton template : HaRP Proxy (HOST)
197+
Daemon Configuration template : HaRP Proxy (HOST)
199198
Surname : appapi-harp
200199
Display name : appapi-harp
201200
Deployment method : docker-install
@@ -208,7 +207,7 @@ Docker network : bridge
208207

209208
Finally, test the whole setup with “Test deploy” in the 3-dots menu of the deploy daemon.
210209

211-
### Additional tests from the network of your hosts (based on Almalinux - RHEL Distros))
210+
### Additional tests from the network of your hosts (based on Almalinux - RHEL Distros)
212211
```
213212
curl -fsS \
214213
-H "harp-shared-key: some_very_secure_password" \

0 commit comments

Comments
 (0)