feat: allow to redefine mimeTypes (possible to use force option) (#349)

Author: gpoitch

README.md

@@ -150,8 +150,15 @@ Type: `Object`
 Default: `null`
 
 This property allows a user to register custom mime types or extension mappings.
-eg. `{ 'text/html': [ 'phtml' ] }`. Please see the documentation for
-[`node-mime`](https://github.com/broofa/node-mime#mimedefine) for more information.
+eg. `mimeTypes: { 'text/html': [ 'phtml' ] }`. 
+
+By default node-mime will throw an error if you try to map a type to an extension 
+that is already assigned to another type. Passing `force: true` will suppress this behavior 
+(overriding any previous mapping).
+eg. `mimeTypes: { typeMap: { 'text/html': [ 'phtml' ] } }, force: true }`. 
+
+Please see the documentation for
+[`node-mime`](https://github.com/broofa/node-mime#mimedefinetypemap-force--false) for more information.
 
 ### publicPath
 

index.js

@@ -38,7 +38,9 @@ module.exports = function wdm(compiler, opts) {
 
   // defining custom MIME type
   if (options.mimeTypes) {
-    mime.define(options.mimeTypes);
+    const typeMap = options.mimeTypes.typeMap || options.mimeTypes;
+    const force = !!options.mimeTypes.force;
+    mime.define(typeMap, force);
   }
 
   const context = createContext(compiler, options);

test/tests/server.js

@@ -245,6 +245,33 @@ describe('Server', () => {
     });
   });
 
+  describe('force option for custom mimeTypes', () => {
+    before((done) => {
+      app = express();
+      const compiler = webpack(webpackClientServerConfig);
+      instance = middleware(compiler, {
+        stats: 'errors-only',
+        logLevel,
+        index: 'Index.phtml',
+        mimeTypes: {
+          typeMap: { 'text/html': ['phtml'] },
+          force: true
+        }
+      });
+      app.use(instance);
+      listen = listenShorthand(done);
+      instance.fileSystem.writeFileSync('/Index.phtml', 'welcome');
+    });
+    after(close);
+
+    it('request to Index.phtml', (done) => {
+      request(app).get('/')
+        .expect('welcome')
+        .expect('Content-Type', /text\/html/)
+        .expect(200, done);
+    });
+  });
+
   describe('WebAssembly', () => {
     before((done) => {
       app = express();