Merge PR ifindev#17: Prevent double https:// in R2 public URLs

Jez · Jez · commit 0e7b9380a29c · 2025-11-08T19:32:25.000+11:00
diff --git a/src/lib/r2.ts b/src/lib/r2.ts
@@ -46,7 +46,8 @@ export async function uploadToR2(
         }
 
         // Return public URL of R2 (should be using custom domain)
-        const publicUrl = `https://${(env as any).CLOUDFLARE_R2_URL}/${key}`;
+        // CLOUDFLARE_R2_URL already includes the protocol (https://)
+        const publicUrl = `${(env as any).CLOUDFLARE_R2_URL}/${key}`;
 
         return {
             success: true,

Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,8 @@ export async function uploadToR2(`
`46`	`46`	`}`
`47`	`47`
`48`	`48`	`// Return public URL of R2 (should be using custom domain)`
`49`		- const publicUrl = `https://${(env as any).CLOUDFLARE_R2_URL}/${key}`;
	`49`	`+ // CLOUDFLARE_R2_URL already includes the protocol (https://)`
	`50`	+ const publicUrl = `${(env as any).CLOUDFLARE_R2_URL}/${key}`;
`50`	`51`
`51`	`52`	`return {`
`52`	`53`	`success: true,`