Added important TODO, should be made a GH issue after 0.13

Author: forman

CHANGES.md

@@ -19,6 +19,8 @@
 
 ### Other
 
+* Intermediate: Added important TODO, should be made a GH issue after 0.13.
+
 * Removed deprecated example `examples/tile`.
 
 ## Changes in 0.13.0.dev10

xcube/server/webservers/tornado.py

@@ -427,6 +427,17 @@ def url_for_path(self,
                      query: Optional[str] = None,
                      reverse: bool = False) -> str:
         """Get the reverse URL for given *path* and *query*."""
+
+        # TODO (forman): in some cases, e.g.,
+        #   when running a tornado server (such as xcube server)
+        #   next to a remote Jupyter Server,
+        #   the URL reported by this implementation is wrong.
+        #   For example with reverse prefix "/proxy/8000", we expect
+        #       https://{host}/users/{user}/proxy/8000/{path}
+        #   but we get
+        #       http://{host}/proxy/8000/{path}
+        #   Also note, that protocol degraded from HTTPS to HTTP!
+        #
         protocol = self._request.protocol
         host = self._request.host
         prefix = self._reverse_url_prefix if reverse else self._url_prefix