diff options
| author | Martin Milata <martin@martinmilata.cz> | 2020-04-03 17:57:34 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-04-03 11:57:34 -0400 |
| commit | b0db928c633ad2e225623cffb20293629c5d5a43 (patch) | |
| tree | 5478d4e177404ec3996f2a946006e8f2364562dd /synapse/app | |
| parent | Add some benchmarks for LruCache (#6446) (diff) | |
| download | synapse-b0db928c633ad2e225623cffb20293629c5d5a43.tar.xz | |
Extend web_client_location to handle absolute URLs (#7006)
Log warning when filesystem path is used. Signed-off-by: Martin Milata <martin@martinmilata.cz>
Diffstat (limited to 'synapse/app')
| -rw-r--r-- | synapse/app/homeserver.py | 16 |
1 files changed, 13 insertions, 3 deletions
diff --git a/synapse/app/homeserver.py b/synapse/app/homeserver.py index f2b56a636f..49df63acd0 100644 --- a/synapse/app/homeserver.py +++ b/synapse/app/homeserver.py @@ -241,16 +241,26 @@ class SynapseHomeServer(HomeServer): resources[SERVER_KEY_V2_PREFIX] = KeyApiV2Resource(self) if name == "webclient": - webclient_path = self.get_config().web_client_location + webclient_loc = self.get_config().web_client_location - if webclient_path is None: + if webclient_loc is None: logger.warning( "Not enabling webclient resource, as web_client_location is unset." ) + elif webclient_loc.startswith("http://") or webclient_loc.startswith( + "https://" + ): + resources[WEB_CLIENT_PREFIX] = RootRedirect(webclient_loc) else: + logger.warning( + "Running webclient on the same domain is not recommended: " + "https://github.com/matrix-org/synapse#security-note - " + "after you move webclient to different host you can set " + "web_client_location to its full URL to enable redirection." + ) # GZip is disabled here due to # https://twistedmatrix.com/trac/ticket/7678 - resources[WEB_CLIENT_PREFIX] = File(webclient_path) + resources[WEB_CLIENT_PREFIX] = File(webclient_loc) if name == "metrics" and self.get_config().enable_metrics: resources[METRICS_PREFIX] = MetricsResource(RegistryProxy) |