Admin: don't escape open project subpath for scheduled reports (#3422)

rilldata · Nov 8, 2023 · 80069de · 80069de
1 parent 03f9bdf
commit 80069de
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/admin/server/server.go b/admin/server/server.go
@@ -395,7 +395,9 @@ func newURLRegistry(opts *Options) *externalURLs {
 }
 
 func (u *externalURLs) reportOpen(org, project, projectSubpath string) string {
-	return urlutil.MustJoinURL(u.frontend, org, project, projectSubpath)
+	res := urlutil.MustJoinURL(u.frontend, org, project)
+	res += projectSubpath // Need to do an unsafe concat to provide flexibility, e.g. to avoid escaping '?'
+	return res
 }
 
 func (u *externalURLs) reportExport(org, project, report string) string {