Make sure dot env is loaded by core app config and as first require when used

Signed-off-by: Alexandre Bodin <bodin.alex@gmail.com>

packages/strapi/lib/Strapi.js

@@ -1,9 +1,7 @@
 'use strict';
 
-// Dependencies.
+// required first because it loads env files.
-const dotenv = require('dotenv');
+const loadConfiguration = require('./core/app-configuration');
-
-dotenv.config({ path: process.env.ENV_PATH });
 
 const http = require('http');
 const path = require('path');
@@ -18,7 +16,6 @@ const { createDatabaseManager } = require('strapi-database');
 
 const utils = require('./utils');
 const loadModules = require('./core/load-modules');
-const loadConfiguration = require('./core/app-configuration');
 const bootstrap = require('./core/bootstrap');
 const initializeMiddlewares = require('./middlewares');
 const initializeHooks = require('./hooks');
@@ -36,7 +33,6 @@ const createTelemetry = require('./services/metrics');
  *
  * @constructor
  */
-
 class Strapi {
   constructor(opts = {}) {
     this.reload = this.reload();

packages/strapi/lib/commands/build.js

@@ -1,11 +1,14 @@
 'use strict';
 
+// required first because it loads env files.
+const loadConfiguration = require('../core/app-configuration');
+
 const { green } = require('chalk');
+
 // eslint-disable-next-line node/no-extraneous-require
 const strapiAdmin = require('strapi-admin');
 const { getConfigUrls } = require('strapi-utils');
 
-const loadConfiguration = require('../core/app-configuration');
 const addSlash = require('../utils/addSlash');
 /**
  * `$ strapi build`

packages/strapi/lib/commands/develop.js

@@ -1,5 +1,8 @@
 'use strict';
 
+// required first because it loads env files.
+const loadConfiguration = require('../core/app-configuration');
+
 const path = require('path');
 const cluster = require('cluster');
 const fs = require('fs-extra');
@@ -7,7 +10,6 @@ const chokidar = require('chokidar');
 const execa = require('execa');
 
 const { logger } = require('strapi-utils');
-const loadConfiguration = require('../core/app-configuration');
 const strapi = require('../index');
 
 /**

packages/strapi/lib/commands/watchAdmin.js

@@ -1,8 +1,12 @@
+'use strict';
+
+// required first because it loads env files.
+const loadConfiguration = require('../core/app-configuration');
+
 // eslint-disable-next-line node/no-extraneous-require
 const strapiAdmin = require('strapi-admin');
 const { getConfigUrls, getAbsoluteServerUrl } = require('strapi-utils');
 
-const loadConfiguration = require('../core/app-configuration');
 const addSlash = require('../utils/addSlash');
 
 module.exports = async function() {

packages/strapi/lib/core/app-configuration/index.js

@@ -1,5 +1,8 @@
 'use strict';
 
+const dotenv = require('dotenv');
+dotenv.config({ path: process.env.ENV_PATH });
+
 process.env.NODE_ENV = process.env.NODE_ENV || 'development';
 
 const os = require('os');