spa routing support

lifeart · lifeart · commit a375bdc30ccd · 2021-09-26T11:56:18.000+03:00
diff --git a/lib/middleware.ts b/lib/middleware.ts
@@ -27,6 +27,16 @@ interface MiddlewareOptions {
   liveReloadPath?: string;
 }
 
+function findClosestIndexFileForPath(outputPath: string, prefix: string): string | undefined {
+  const candidates = [];
+  const parts = prefix.split('/');
+  while (parts.length) {
+    parts.pop();
+    candidates.push(resolvePath(outputPath, [...parts, 'index.html'].join(path.sep)));
+  }
+  return candidates.find(file => fs.existsSync(file));
+}
+
 // You must call watcher.start() before you call `getMiddleware`
 //
 // This middleware is for development use only. It hasn't been reviewed
@@ -45,7 +55,7 @@ function handleRequest(
   // eslint-disable-next-line node/no-deprecated-api
   const urlObj = url.parse(request.url);
   const pathname = urlObj.pathname || '';
-  let filename: string, stat;
+  let filename: string, stat!: fs.Stats;
 
   try {
     filename = decodeURIComponent(pathname);
@@ -66,9 +76,24 @@ function handleRequest(
   try {
     stat = fs.statSync(filename);
   } catch (e) {
-    // not found
-    next();
-    return;
+    const nameStats = path.parse(filename);
+    const maybeIndex = findClosestIndexFileForPath(outputPath, filename.substr(1));
+
+    // if it's looks like an SPA path
+    if (nameStats.ext === '' && maybeIndex) {
+      filename = maybeIndex.replace(path.sep + 'index.html', '');
+      try {
+        stat = fs.statSync(filename);
+      } catch (e) {
+        // not found
+        next();
+        return;
+      }
+    } else {
+      // not found
+      next();
+      return;
+    }
   }
 
   if (stat.isDirectory()) {
diff --git a/test/builder_test.js b/test/builder_test.js
@@ -1040,7 +1040,7 @@ describe('Builder', function() {
         // the actual results of process.hrtime() are not
         // reliable
         if (process.env.CI !== 'true') {
-          expect(a).to.be.within(b, b + 10e6);
+          expect(a).to.be.within(b, b + 15e6);
         }
       };
 
diff --git a/test/fixtures/spa/index.html b/test/fixtures/spa/index.html
@@ -0,0 +1,3 @@
+<html>
+    <body>Hello from SPA</body>
+</html>
diff --git a/test/server_test.js b/test/server_test.js
@@ -113,6 +113,37 @@ describe('server', function() {
     expect(statusCode).to.eql(200);
   }).timeout(5000);
 
+  it('support SPA routing to index.html from child paths', async function() {
+    const mockUI = new MockUI();
+    const builder = new Builder(new broccoliSource.WatchedDir('test/fixtures/spa'));
+    const watcher = new Watcher(builder, []);
+    server = new Server.Server(watcher, '127.0.0.1', PORT, undefined, mockUI);
+    server.start();
+    await new Promise(resolve => {
+      server.instance.on('listening', resolve);
+    });
+    const { statusCode, body } = await got(`http://127.0.0.1:${PORT}/foo/bar/baz`); // basic serving
+    expect(statusCode).to.eql(200);
+    expect(body).to.contain('Hello from SPA');
+  }).timeout(5000);
+
+  it("skip SPA routing to index.html from child path if it's ends with extension", async function() {
+    const mockUI = new MockUI();
+    const builder = new Builder(new broccoliSource.WatchedDir('test/fixtures/spa'));
+    const watcher = new Watcher(builder, []);
+    server = new Server.Server(watcher, '127.0.0.1', PORT, undefined, mockUI);
+    server.start();
+    await new Promise(resolve => {
+      server.instance.on('listening', resolve);
+    });
+    try {
+      await got(`http://127.0.0.1:${PORT}/foo/bar/baz.png`);
+      expect.fail('expected rejection');
+    } catch (e) {
+      expect(e.body).to.include(`Cannot GET /foo/bar/baz.png`);
+    }
+  }).timeout(5000);
+
   it('buildSuccess is handled', async function() {
     const mockUI = new MockUI();
     const builder = new Builder(new broccoliSource.WatchedDir('test/fixtures/basic'));

Original file line number	Diff line number	Diff line change
`@@ -1040,7 +1040,7 @@ describe('Builder', function() {`
`1040`	`1040`	`// the actual results of process.hrtime() are not`
`1041`	`1041`	`// reliable`
`1042`	`1042`	`if (process.env.CI !== 'true') {`
`1043`		`- expect(a).to.be.within(b, b + 10e6);`
	`1043`	`+ expect(a).to.be.within(b, b + 15e6);`
`1044`	`1044`	`}`
`1045`	`1045`	`};`
`1046`	`1046`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+<html>`
	`2`	`+ <body>Hello from SPA</body>`
	`3`	`+</html>`