Added process.browser to DefinePlugin

Cowboy-coder · Cowboy-coder · commit aa0fc5bfbcc8 · 2017-05-15T00:20:24.000+02:00
The idea is that there is no need for passing

    if (!process.browser) {
      console.log('hello from server!');
    }

to the client side bundle.

This can also be used for storing server-side-only secrets without
leaking it to the client. Although i'm not sure if that's a good idea,
because accidentally running the app in `NODE_ENV==development` in
"production" would leak all those secrets.
diff --git a/server/build/webpack.js b/server/build/webpack.js
@@ -112,7 +112,8 @@ export default async function createCompiler (dir, { dev = false, quiet = false,
       filename: 'manifest.js'
     }),
     new webpack.DefinePlugin({
-      'process.env.NODE_ENV': JSON.stringify(dev ? 'development' : 'production')
+      'process.env.NODE_ENV': JSON.stringify(dev ? 'development' : 'production'),
+      'process.browser': JSON.stringify(true)
     }),
     new PagesPlugin(),
     new CaseSensitivePathPlugin()
