about summary refs log tree commit diff
path: root/ops
diff options
context:
space:
mode:
authorVincent Ambo <mail@tazj.in>2021-12-19T16·37+0300
committertazjin <mail@tazj.in>2021-12-19T16·46+0000
commitf8b4029b1744bea77cc3f941d59c09d11f714f47 (patch)
tree3d65a89afd20730045afc036acb91424e275b012 /ops
parent3a2a5ffa9d77e12960ba0349e1810efb068f0701 (diff)
fix(ops/besadii): Stop path.Join from eating our URL r/3305
apparently this chomps away at things inside of fragment strings

Change-Id: Ie60d52d101dc4281b3a62c228af076791e1c7928
Reviewed-on: https://cl.tvl.fyi/c/depot/+/4462
Tested-by: BuildkiteCI
Reviewed-by: grfn <grfn@gws.fyi>
Diffstat (limited to 'ops')
-rw-r--r--ops/besadii/main.go2
1 files changed, 1 insertions, 1 deletions
diff --git a/ops/besadii/main.go b/ops/besadii/main.go
index 5c9baf6bf530..b382821bcce7 100644
--- a/ops/besadii/main.go
+++ b/ops/besadii/main.go
@@ -160,7 +160,7 @@ func loadConfig() (*config, error) {
 
 // linkToChange creates the full link to a change's patchset in Gerrit
 func linkToChange(cfg *config, changeId, patchset string) string {
-	return path.Join(cfg.GerritUrl, "c", cfg.Repository, "+", changeId, patchset)
+	return fmt.Sprintf("%s/c/%s/+/%s/%s", cfg.GerritUrl, "c", cfg.Repository, "+", changeId, patchset)
 }
 
 // updateGerrit posts a comment on a Gerrit CL to indicate the current build status.