Update README.md - Instruction for Apache reverse proxy and 3 hosts#89
Update README.md - Instruction for Apache reverse proxy and 3 hosts#89manik0u wants to merge 1 commit intonextcloud:mainfrom
Conversation
Update of the Readme related to this issue : nextcloud#85 Signed-off-by: Manikou <91145906+manik0u@users.noreply.github.com>
oleksandr-nc
left a comment
oleksandr-nc
left a comment
There was a problem hiding this comment.
Can we consider placing it after "Configuring Your Reverse Proxy" or in a new dedicated section.
Sorry for so late review.
Will you be fine if I open a PR replacing it based on your PR? Or I can edit this PR to not lost that idea comes from you.
| ### On the Nextcloud Web Interface - Daemon Register | ||
| Add the following configuration : | ||
| ``` | ||
| Daemon Configuraiton template : HaRP Proxy (HOST) |
There was a problem hiding this comment.
Should be "Configuration"
| # AppAPI Configuration | ||
| ProxyPass /exapps/ http://<IP_host2_docker>:8780/exapps/ | ||
| ProxyPassReverse /exapps/ http://<IP_host2_docker>:8780/exapps/ | ||
| ``` |
There was a problem hiding this comment.
Can we add ProxyTimeout 1800 here?
|
|
||
| ##### On the Docker Host | ||
| ##### Creation of Cert folder (if necessary) | ||
| `mkdir -p /some/path/{certs,}` |
There was a problem hiding this comment.
For which cases do we need the empty-name expansion?
| -d ghcr.io/nextcloud/nextcloud-appapi-harp:release | ||
| ``` | ||
|
|
||
| > **Warning:** Do not forget to change the **HP_SHARED_KEY** value to a secure one! |
There was a problem hiding this comment.
A second "Warning: Do not forget to change HP_SHARED_KEY" that's redundant - it already exists immediately above. (line 74)
|
|
||
| Finally, test the whole setup with “Test deploy” in the 3-dots menu of the deploy daemon. | ||
|
|
||
| ### Additional tests from the network of your hosts (based on Almalinux - RHEL Distros)) |
There was a problem hiding this comment.
I guess second ) is a typo?
|
Hi @oleksandr-nc , Sorry for the delay in responding. |
2 participants
Update of the Readme related to this issue : #85