diff options
| author | Vincent Ambo <mail@tazj.in> | 2021-12-19T16·37+0300 |
|---|---|---|
| committer | tazjin <mail@tazj.in> | 2021-12-19T16·46+0000 |
| commit | f8b4029b1744bea77cc3f941d59c09d11f714f47 (patch) | |
| tree | 3d65a89afd20730045afc036acb91424e275b012 /ops | |
| parent | 3a2a5ffa9d77e12960ba0349e1810efb068f0701 (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.go | 2 |
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. |