LNT: Use relative URLs for Javascript-based URLs.

georges-arm · TamarChristinaArm · commit 51cae4e78379 · 2020-12-12T16:27:48.000Z
Having _external=True seems to break links when LNT is hosted behind a reverse-proxy. This has been observed previously as https://reviews.llvm.org/D85425 for example. Reviewed By: thopre Differential Revision: https://reviews.llvm.org/D93116
diff --git a/lnt/server/ui/templates/layout.html b/lnt/server/ui/templates/layout.html
@@ -3,7 +3,7 @@
 <head>
     <meta charset="UTF-8"/>
 
-    <script>var lnt_url_base="{{ url_for('.index', _external=True) }}".replace(/\/$/, "");</script>
+    <script>var lnt_url_base="{{ url_for('.index', _external=False) }}".replace(/\/$/, "");</script>
     <link href="{{ url_for('.static', filename='bootstrap/css/bootstrap.min.css')
                 }}" rel="stylesheet" media="screen"/>
     <link href="{{ url_for('.static', filename='bootstrap/css/bootstrap-responsive.min.css')
