fix: only stub process.env if not targeting node

src/index.js

@@ -32,8 +32,14 @@ class Dotenv {
  }, config)
 
  this.checkDeprecation()
+ }
+
+ apply (compiler) {
+ const target = compiler.options.target ?? 'web'
+ const variables = this.gatherVariables()
+ const data = this.formatData(variables, target)
 
- return new DefinePlugin(this.formatData(this.gatherVariables()))
+ new DefinePlugin(data).apply(compiler)
  }
 
  checkDeprecation () {
@@ -121,7 +127,7 @@ class Dotenv {
  return ''
  }
 
- formatData (vars = {}) {
+ formatData (vars = {}, target) {
  const { expand } = this.config
  const formatted = Object.keys(vars).reduce((obj, key) => {
  const v = vars[key]
@@ -144,8 +150,14 @@ class Dotenv {
  return obj
  }, {})
 
- // fix in case of missing
- formatted['process.env'] = '{}'
+ // We have to stub any remaining `process.env`s due to Webpack 5 not polyfilling it anymore
+ // https://github.com/mrsteele/dotenv-webpack/issues/240#issuecomment-710231534
+ // However, if someone targets Node or Electron `process.env` still exists, and should therefore be kept
+ // https://webpack.js.org/configuration/target
+ if (!target.startsWith('node') && !target.startsWith('electron')) {
+ // fix in case of missing
+ formatted['process.env'] = '{}'
+ }
 
  return formatted
  }