Merge pull request #574 from albedosehen/fix/web-server-mime-types

Expand MIME type map in web server to broaden asset type support with fallback (fonts, source maps, etc)

Author: nobkd

packages/nuekit/src/nueserver.js

@@ -10,18 +10,29 @@ import http from 'node:http'
 import { join, extname } from 'node:path'
 
 export const TYPES = {
-  html: 'text/html; charset=UTF-8',
-  js:   'application/javascript',
-  wasm: 'application/wasm',
-  json: 'application/json',
-  svg:  'image/svg+xml',
-  ico:  'image/x-icon',
-  webp: 'image/webp',
-  png:  'image/png',
-  jpeg: 'image/jpg',
-  jpg:  'image/jpg',
-  css:  'text/css',
-  csv:  'text/csv',
+  html:  'text/html; charset=UTF-8',
+  js:    'application/javascript',
+  wasm:  'application/wasm',
+  json:  'application/json',
+  svg:   'image/svg+xml',
+  ico:   'image/x-icon',
+  webp:  'image/webp',
+  png:   'image/png',
+  jpeg:  'image/jpeg',
+  jpg:   'image/jpeg',
+  txt:   'text/plain',
+  css:   'text/css',
+  csv:   'text/csv',
+  woff:  'font/woff',
+  woff2: 'font/woff2',
+  ttf:   'font/ttf',
+  otf:   'font/otf',
+  eot:   'application/vnd.ms-fontobject',
+  mp4:   'video/mp4',
+  webm:  'video/webm',
+  mp3:   'audio/mpeg',
+  ogg:   'audio/ogg',
+  default: 'application/octet-stream'
 }
 
 let sessions = []
@@ -53,7 +64,9 @@ export function createServer(root, callback) {
       const { code, path } = !ext || ext == 'html' ? await callback(url, _) : { path: url }
       if (!path) throw { errno: -2 }
       const buffer = await fs.readFile(join(root, path))
-      res.writeHead(code || 200, { 'content-type': TYPES[ext] })
+      res.writeHead(code || 200, {
+        'content-type': TYPES[ext] || TYPES.default
+      })      
       res.end(buffer)
 
     } catch (e) {

packages/nuekit/test/misc.test.js

@@ -19,7 +19,9 @@ beforeEach(async () => {
   await fs.mkdir(root, { recursive: true })
 })
 
-afterEach(async () => await fs.rm(root, { recursive: true, force: true }))
+afterEach(async () => {
+  await fs.rm(root, { recursive: true, force: true })
+})
 
 async function write(filename, code) {
   const file = join(root, filename)

packages/nuekit/test/nueserver.test.js

@@ -0,0 +1,46 @@
+import { promises as fs } from 'node:fs'
+import { join } from 'node:path'
+
+import { createServer, TYPES } from '../src/nueserver.js'
+
+// temporary directory
+const root = '_test'
+const PORT = 3403
+const HOST = `http://localhost:${PORT}`
+const testFiles = []
+
+
+// setup and teardown
+beforeAll(async () => {
+  await fs.rm(root, { recursive: true, force: true })
+  await fs.mkdir(root, { recursive: true })
+
+  // Setup: Create a temporary directory & a test file for each MIME type
+  for (const ext in TYPES) {
+    const filename = `test.${ext == 'default' ? 'unknown' : ext}`
+    const filePath = join(root, filename)
+
+    await fs.writeFile(filePath, '')
+    testFiles.push({ ext, filename, filePath })
+  }
+})
+
+afterAll(async () => {
+  await fs.rm(root, { recursive: true, force: true })
+})
+
+
+test('serve all file extensions with correct MIME type', async () => {
+  const server = createServer(root, (path) => ({ path }))
+  server.listen(PORT)
+
+  try {
+    for (const { ext, filename } of testFiles) {
+      const res = await fetch(`${HOST}/${filename}`)
+      const contentType = res.headers.get('content-type')
+
+      expect(res.status).toBe(200)
+      expect(contentType).toBe(TYPES[ext])
+    }
+  } finally { server.close() }
+})