From d76433caeb5d107088925bfd17d04034446cda09 Mon Sep 17 00:00:00 2001 From: Phil Date: Thu, 10 Aug 2023 20:48:26 -0400 Subject: [PATCH] Fix azure pre-signed urls Azure URL signing was broken. The journal prefix was being duplicated in the URL, which resulted in unusable URLs. Clients would fail with a 403 response when attempting to fetch the fragment. This removes the extra prefix to hopefully allow signing to work. --- broker/fragment/store_azure.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/broker/fragment/store_azure.go b/broker/fragment/store_azure.go index 10509038..308d94a9 100644 --- a/broker/fragment/store_azure.go +++ b/broker/fragment/store_azure.go @@ -82,7 +82,7 @@ func (a *azureBackend) SignGet(ep *url.URL, fragment pb.Fragment, d time.Duratio if err != nil { return "", err } - return fmt.Sprintf("%s/%s/%s?%s", cfg.containerURL(), cfg.prefix, blobName, sasQueryParams.Encode()), nil + return fmt.Sprintf("%s/%s?%s", cfg.containerURL(), blobName, sasQueryParams.Encode()), nil } func (a *azureBackend) Exists(ctx context.Context, ep *url.URL, fragment pb.Fragment) (bool, error) {