Merge pull request #25707 from github/repo-sync

repo sync

middleware/handle-invalid-paths.js

@@ -31,5 +31,10 @@ export default function handleInvalidPaths(req, res, next) {
     return res.status(404).send('Not found')
   }
 
+  if (req.path.endsWith('/index.md')) {
+    defaultCacheControl(res)
+    return res.redirect(req.path.replace(/\/index\.md$/, ''))
+  }
+
   return next()
 }

tests/rendering-fixtures/bad-urls.js

@@ -0,0 +1,14 @@
+import { head } from '../helpers/e2etest.js'
+
+describe('bad URLs', () => {
+  test('any URL with /index.md suffix redirects to be without suffix', async () => {
+    const URLs = ['/index.md', '/en/index.md', '/en/actions/index.md']
+    for (const url of URLs) {
+      const res = await head(url)
+      expect(res.statusCode).toBe(302)
+      expect(res.headers.location).toBe(url.replace(/\/index\.md$/, ''))
+      expect(res.headers['cache-control']).toContain('public')
+      expect(res.headers['cache-control']).toMatch(/max-age=[1-9]/)
+    }
+  })
+})