Solve Types problems

Author: gualtierim

src/api/middlewares/AuthenticateMiddleware.ts

@@ -1,19 +1,17 @@
-import { inject, named } from 'inversify';
+import {inject, named} from 'inversify';
 import * as Request from 'request';
-import { Logger as LoggerType } from '../../core/Logger';
-import { Types, Core } from '../../constants';
-import { events } from '../../core/api/events';
-import { UserAuthenticatedListener } from '../listeners/user/UserAuthenticatedListener';
+import {Logger as LoggerType} from '../../core/Logger';
+import {Types, Core} from '../../constants';
+import {events} from '../../core/api/events';
+import {UserAuthenticatedListener} from '../listeners/user/UserAuthenticatedListener';
 
 
 export class AuthenticateMiddleware implements interfaces.Middleware {
 
     public log: LoggerType;
 
-    constructor(
-        @inject(Types.Core) @named(Core.Logger) Logger: typeof LoggerType,
-        @inject(Types.Lib) @named('request') private request: typeof Request
-    ) {
+    constructor(@inject(Types.Core) @named(Core.Logger) Logger: typeof LoggerType,
+                @inject(Types.Lib) @named('request') private request: typeof Request) {
         this.log = new Logger(__filename);
     }
 
@@ -57,7 +55,7 @@ export class AuthenticateMiddleware implements interfaces.Middleware {
     }
 
     private getToken(req: myExpress.Request): string | null {
-        const authorization = req.headers.authorization;
+        const authorization: string = req.headers.authorization as string;
 
         // Retrieve the token form the Authorization header
         if (authorization && authorization.split(' ')[0] === 'Bearer') {

src/config/Database.ts

@@ -16,8 +16,8 @@ export const DatabaseConfig = {
     client: process.env.DB_CLIENT,
     connection: process.env.DB_CONNECTION,
     pool: {
-        min: parseInt(process.env.DB_POOL_MIN, 10),
-        max: parseInt(process.env.DB_POOL_MAX, 10)
+        min: parseInt(process.env.DB_POOL_MIN as string, 10),
+        max: parseInt(process.env.DB_POOL_MAX as string, 10)
     },
     migrations: {
         directory: process.env.DB_MIGRATION_DIR,

src/config/LoggerConfig.ts

@@ -13,6 +13,6 @@ import { Configurable } from '../core/App';
 export class LoggerConfig implements Configurable {
     public configure(): void {
         Logger.addAdapter('winston', WinstonAdapter);
-        Logger.setAdapter(process.env.LOG_ADAPTER);
+        Logger.setAdapter(process.env.LOG_ADAPTER as string);
     }
 }

src/core/ApiInfo.ts

@@ -7,11 +7,11 @@ import { ApiMonitor } from './ApiMonitor';
 export class ApiInfo {
 
     public static getRoute(): string {
-        return process.env.APP_URL_PREFIX + process.env.API_INFO_ROUTE;
+        return process.env.APP_URL_PREFIX as string + process.env.API_INFO_ROUTE;
     }
 
     public setup(application: express.Application): void {
-        if (Environment.isTruthy(process.env.API_INFO_ENABLED)) {
+        if (Environment.isTruthy(process.env.API_INFO_ENABLED as string)) {
             application.get(
                 ApiInfo.getRoute(),
                 // @ts-ignore: False type definitions from express
@@ -20,11 +20,11 @@ export class ApiInfo {
                     const links = {
                         links: {}
                     };
-                    if (Environment.isTruthy(process.env.SWAGGER_ENABLED)) {
+                    if (Environment.isTruthy(process.env.SWAGGER_ENABLED as string)) {
                         links.links['swagger'] =
                             `${application.get('host')}${SwaggerUI.getRoute()}`;
                     }
-                    if (Environment.isTruthy(process.env.MONITOR_ENABLED)) {
+                    if (Environment.isTruthy(process.env.MONITOR_ENABLED as string)) {
                         links.links['monitor'] =
                             `${application.get('host')}${ApiMonitor.getRoute()}`;
                     }

src/core/ApiMonitor.ts

@@ -7,11 +7,11 @@ import { BasicAuthentication } from './BasicAuthentication';
 export class ApiMonitor {
 
     public static getRoute(): string {
-        return process.env.MONITOR_ROUTE;
+        return process.env.MONITOR_ROUTE as string;
     }
 
     public setup(app: express.Application): void {
-        if (Environment.isTruthy(process.env.MONITOR_ENABLED)) {
+        if (Environment.isTruthy(process.env.MONITOR_ENABLED as string)) {
             app.use(monitor());
             app.get(ApiMonitor.getRoute(), BasicAuthentication(), monitor().pageRoute);
         }

src/core/BasicAuthentication.ts

@@ -4,7 +4,7 @@ import * as basicAuth from 'express-basic-auth';
 export const BasicAuthentication = (): any => {
     return basicAuth({
         users: {
-            [process.env.APP_BASIC_USER]: process.env.APP_BASIC_PASSWORD
+            [process.env.APP_BASIC_USER as string]: process.env.APP_BASIC_PASSWORD as string
         },
         challenge: true
     });

src/core/Bootstrap.ts

@@ -40,7 +40,7 @@ export class Bootstrap {
 
     public setupInversifyExpressServer(app: express.Application, ioc: IoC): InversifyExpressServer {
         const inversifyExpressServer = new InversifyExpressServer(ioc.container, undefined, {
-            rootPath: process.env.APP_URL_PREFIX
+            rootPath: process.env.APP_URL_PREFIX as string
         }, app);
         // @ts-ignore: False type definitions from express
         inversifyExpressServer.setConfig((a) => a.use(extendExpressResponse));

src/core/Server.ts

@@ -72,13 +72,13 @@ export class Server {
         this.log.debug(`Environment  : ${Environment.getNodeEnv()}`);
         this.log.debug(`Version      : ${Environment.getPkg().version}`);
         this.log.debug(``);
-        if (Environment.isTruthy(process.env.API_INFO_ENABLED)) {
+        if (Environment.isTruthy(process.env.API_INFO_ENABLED as string)) {
             this.log.debug(`API Info     : ${app.get('host')}${ApiInfo.getRoute()}`);
         }
-        if (Environment.isTruthy(process.env.SWAGGER_ENABLED)) {
+        if (Environment.isTruthy(process.env.SWAGGER_ENABLED as string)) {
             this.log.debug(`Swagger      : ${app.get('host')}${SwaggerUI.getRoute()}`);
         }
-        if (Environment.isTruthy(process.env.MONITOR_ENABLED)) {
+        if (Environment.isTruthy(process.env.MONITOR_ENABLED as string)) {
             this.log.debug(`Monitor      : ${app.get('host')}${ApiMonitor.getRoute()}`);
         }
         this.log.debug('-------------------------------------------------------');

src/core/SwaggerUI.ts

@@ -8,14 +8,14 @@ import { BasicAuthentication } from './BasicAuthentication';
 export class SwaggerUI {
 
     public static getRoute(): string {
-        return process.env.SWAGGER_ROUTE;
+        return process.env.SWAGGER_ROUTE as string;
     }
 
     public setup(app: express.Application): void {
-        if (Environment.isTruthy(process.env.SWAGGER_ENABLED)) {
+        if (Environment.isTruthy(process.env.SWAGGER_ENABLED as string)) {
             const baseFolder = __dirname.indexOf(`${path.sep}src${path.sep}`) >= 0 ? `${path.sep}src${path.sep}` : `${path.sep}dist${path.sep}`;
             const basePath = __dirname.substring(0, __dirname.indexOf(baseFolder));
-            const swaggerFile = require(path.join(basePath, process.env.SWAGGER_FILE));
+            const swaggerFile = require(path.join(basePath, process.env.SWAGGER_FILE as string));
             const packageJson = require(path.join(basePath, 'package.json'));
 
             // Add npm infos to the swagger doc