Skip to content

Commit 84bd234

Browse files
committed
fix(stdiscosrv): remove unused ssl headers when proxying
1 parent 5bb28ea commit 84bd234

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

users/stdiscosrv.rst

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -218,6 +218,8 @@ the Syncthing settings.
218218
proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
219219
proxy_set_header X-Forwarded-Proto $http_x_forwarded_proto;
220220
proxy_set_header X-SSL-Cert $ssl_client_cert;
221+
proxy_set_header X-Forwarded-Tls-Client-Cert "";
222+
proxy_set_header X-Tls-Client-Cert-Der-Base64 "";
221223
upstream discovery.example.com {
222224
# Local IP address:port for discovery server
223225
server 192.0.2.1:8443;
@@ -278,6 +280,8 @@ The following lines must be added to the configuration:
278280
SSLProxyEngine On
279281
SSLVerifyClient optional_no_ca
280282
RequestHeader set X-SSL-Cert "%{SSL_CLIENT_CERT}s"
283+
RequestHeader unset X-Forwarded-Tls-Client-Cert
284+
RequestHeader unset X-Tls-Client-Cert-Der-Base64
281285
282286
The following was observed to not be required at least under
283287
Apache httpd 2.4.38, as the proxy module adds the needed header by default.
@@ -300,6 +304,8 @@ The following lines must be added to the Caddyfile:
300304
header_up X-Forwarded-For {http.request.remote.host}
301305
header_up X-Client-Port {http.request.remote.port}
302306
header_up X-Tls-Client-Cert-Der-Base64 {http.request.tls.client.certificate_der_base64}
307+
header_up -X-Ssl-Cert
308+
header_up -X-Forwarded-Tls-Client-Cert
303309
}
304310
305311
tls {

0 commit comments

Comments
 (0)