fix: use middleware for catch-all route in Express 5

A simple tool which lets you scrape twitter accounts and crosspost them to bluesky accounts! Comes with a CLI and a webapp for managing profiles! Works with images/videos/link embeds/threads.

fork atom

Express 5's path-to-regexp does not support wildcard patterns like * or /*.
Using app.use() as a fallback middleware properly serves the frontend
when no API route matches.

jack 2 months ago 463d5374 b6562a45

+2 -2

1 changed file

expand all

unified split

src

server.ts

+2 -2

src/server.ts

··· 114 res.json({ success: true }); 115 }); 116 117 - // Serve the frontend for any other route 118 - app.get('/*', (_req, res) => { 119 res.sendFile(path.join(__dirname, '../public/index.html')); 120 }); 121

··· 114 res.json({ success: true }); 115 }); 116 117 + // Serve the frontend for any other route (middleware approach for Express 5) 118 + app.use((_req, res) => { 119 res.sendFile(path.join(__dirname, '../public/index.html')); 120 }); 121