Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Commit 051ddac

Browse files
authored
Clarifications to reverse_proxy.md (#10708)
* Update reverse_proxy.md * Create 10708.doc
1 parent 029b7ad commit 051ddac

File tree

2 files changed

+6
-5
lines changed

2 files changed

+6
-5
lines changed

changelog.d/10708.doc

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Minor clarifications to the documentation for reverse proxies.

docs/reverse_proxy.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,9 @@ server {
6464
server_name matrix.example.com;
6565
6666
location ~* ^(\/_matrix|\/_synapse\/client) {
67+
# note: do not add a path (even a single /) after the port in `proxy_pass`,
68+
# otherwise nginx will canonicalise the URI and cause signature verification
69+
# errors.
6770
proxy_pass http://localhost:8008;
6871
proxy_set_header X-Forwarded-For $remote_addr;
6972
proxy_set_header X-Forwarded-Proto $scheme;
@@ -76,10 +79,7 @@ server {
7679
}
7780
```
7881

79-
**NOTE**: Do not add a path after the port in `proxy_pass`, otherwise nginx will
80-
canonicalise/normalise the URI.
81-
82-
### Caddy 1
82+
### Caddy v1
8383

8484
```
8585
matrix.example.com {
@@ -99,7 +99,7 @@ example.com:8448 {
9999
}
100100
```
101101

102-
### Caddy 2
102+
### Caddy v2
103103

104104
```
105105
matrix.example.com {

0 commit comments

Comments
 (0)