Merge pull request #159 from espadonne/main

add server-side audio proxy for S3 media files

Author: mfwolffe

pages/api/audio-proxy.js

@@ -0,0 +1,50 @@
+/**
+ * Server-side audio proxy for S3-hosted media files.
+ *
+ * In production the Django backend stores media on S3 and returns pre-signed
+ * S3 URLs.  Browser-side fetches (especially from Web Workers) fail because
+ * the S3 bucket has no CORS policy for the frontend origin.  This route
+ * fetches on the server (Cloudflare Worker / Node dev server) where CORS
+ * does not apply and streams the result back same-origin.
+ */
+export default async function handler(req, res) {
+  if (req.method !== 'GET') {
+    res.setHeader('Allow', 'GET');
+    return res.status(405).end();
+  }
+
+  const { url } = req.query;
+  if (!url) return res.status(400).json({ error: 'Missing url parameter' });
+
+  // Only allow known-safe destinations
+  let parsed;
+  try {
+    parsed = new URL(url);
+  } catch {
+    return res.status(400).json({ error: 'Invalid URL' });
+  }
+
+  const allowed =
+    parsed.hostname.endsWith('.amazonaws.com') ||
+    parsed.hostname.endsWith('.musiccpr.org') ||
+    parsed.hostname === 'localhost';
+  if (!allowed) return res.status(403).json({ error: 'Domain not allowed' });
+
+  try {
+    const upstream = await fetch(url);
+    if (!upstream.ok) return res.status(upstream.status).end();
+
+    const arrayBuffer = await upstream.arrayBuffer();
+
+    res.setHeader(
+      'Content-Type',
+      upstream.headers.get('content-type') || 'audio/mpeg',
+    );
+    res.setHeader('Content-Length', arrayBuffer.byteLength);
+    res.setHeader('Cache-Control', 'public, max-age=3600');
+    res.send(Buffer.from(arrayBuffer));
+  } catch (err) {
+    console.error('audio-proxy: upstream fetch failed', err);
+    res.status(502).json({ error: 'Upstream fetch failed' });
+  }
+}

pages/courses/[slug]/[piece]/[actCategory]/[partType]/activity/[step].js

@@ -285,11 +285,11 @@ export default function ActivityPage() {
   const basslineAssignment = loadedActivities
     ? activities[piece]?.find((a) => a.part_type === 'Bassline')
     : null;
-  // Convert absolute backend URL to relative /media/... path so it routes through
-  // Next.js rewrite proxy (avoids CORS issues with cross-origin fetch/decodeAudioData)
+  // Proxy S3 URLs through our API route so the browser (and Web Workers) can
+  // fetch same-origin without needing S3 CORS configuration.
   const rawBasslineURL = basslineAssignment?.part?.sample_audio || null;
   const basslineURL = rawBasslineURL
-    ? rawBasslineURL.replace(/^https?:\/\/[^/]+/, '') // strip origin, keep /media/...
+    ? `/api/audio-proxy?url=${encodeURIComponent(rawBasslineURL)}`
     : null;
   const sampleTakes = (stepNumber === 3 && basslineURL) ? [{
     id: 'sample-bassline',