@foal/core - npm Package Compare versions

Comparing version 0.6.0-alpha.3 to 0.6.0-alpha.4

lib/auth/authentication/login.controller.js

@@ -21,3 +21,3 @@ "use strict";
 exports.strategy = strategy;
-class LoginController {
+let LoginController = class LoginController {
     constructor(services) {
@@ -53,3 +53,3 @@ this.services = services;
     }
-}
+};
 __decorate([
@@ -67,3 +67,7 @@ core_1.Get('/logout'),
 ], LoginController.prototype, "login", null);
+LoginController = __decorate([
+    core_1.Controller(),
+    __metadata("design:paramtypes", [core_1.ServiceManager])
+], LoginController);
 exports.LoginController = LoginController;
 //# sourceMappingURL=login.controller.js.map

lib/auth/authorization/permission-required.hook.d.ts

 import { HookDecorator } from '../../core';
-export declare function PermissionRequired(perm: string): HookDecorator;
+export declare function PermissionRequired(perm: string, options?: {
+    redirect?: string;
+}): HookDecorator;

lib/auth/authorization/permission-required.hook.js

 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 const core_1 = require("../../core");
-function PermissionRequired(perm) {
+function PermissionRequired(perm, options = {}) {
     return core_1.Hook(ctx => {
         if (!ctx.user) {
+            if (options.redirect) {
+                return new core_1.HttpResponseRedirect(options.redirect);
+            }
             return new core_1.HttpResponseUnauthorized();
@@ -8,0 +11,0 @@ }

lib/core/controllers.d.ts

 import 'reflect-metadata';
+import { Class } from './class.interface';
+import { ServiceManager } from './service-manager';
 export declare function Controller(path?: string): {
@@ -6,1 +8,2 @@ (target: Function): void;
 };
+export declare function createController<T>(controllerClass: Class<T>, services?: ServiceManager): T;

lib/core/controllers.js

@@ -5,2 +5,4 @@ "use strict";
 require("reflect-metadata");
+const utils_1 = require("./routes/utils");
+const service_manager_1 = require("./service-manager");
 function Controller(path) {
@@ -10,2 +12,10 @@ return Reflect.metadata('path', path);
 exports.Controller = Controller;
+function createController(controllerClass, services) {
+    const controllerDependencies = utils_1.getMetadata('design:paramtypes', controllerClass) || [];
+    if (!services) {
+        services = new service_manager_1.ServiceManager();
+    }
+    return new controllerClass(...controllerDependencies.map(serviceClass => services.get(serviceClass)));
+}
+exports.createController = createController;
 //# sourceMappingURL=controllers.js.map

lib/core/http/http-responses.d.ts

@@ -11,3 +11,3 @@ export declare abstract class HttpResponse {
 }
-export declare function isHttpResponse(obj: any): boolean;
+export declare function isHttpResponse(obj: any): obj is HttpResponse;
 export declare abstract class HttpResponseSuccess extends HttpResponse {
@@ -17,3 +17,3 @@ readonly isHttpResponseSuccess: boolean;
 }
-export declare function isHttpResponseSuccess(obj: any): boolean;
+export declare function isHttpResponseSuccess(obj: any): obj is HttpResponseSuccess;
 export declare class HttpResponseOK extends HttpResponseSuccess {
@@ -25,3 +25,3 @@ readonly isHttpResponseOK: boolean;
 }
-export declare function isHttpResponseOK(obj: any): boolean;
+export declare function isHttpResponseOK(obj: any): obj is HttpResponseOK;
 export declare class HttpResponseCreated extends HttpResponseSuccess {
@@ -33,3 +33,3 @@ readonly isHttpResponseCreated: boolean;
 }
-export declare function isHttpResponseCreated(obj: any): boolean;
+export declare function isHttpResponseCreated(obj: any): obj is HttpResponseCreated;
 export declare class HttpResponseNoContent extends HttpResponseSuccess {
@@ -41,3 +41,3 @@ readonly isHttpResponseNoContent: boolean;
 }
-export declare function isHttpResponseNoContent(obj: any): boolean;
+export declare function isHttpResponseNoContent(obj: any): obj is HttpResponseNoContent;
 export declare abstract class HttpResponseRedirection extends HttpResponse {
@@ -47,3 +47,3 @@ readonly isHttpResponseRedirection: boolean;
 }
-export declare function isHttpResponseRedirection(obj: any): boolean;
+export declare function isHttpResponseRedirection(obj: any): obj is HttpResponseRedirection;
 export declare class HttpResponseRedirect extends HttpResponseRedirection {
@@ -56,3 +56,3 @@ path: string;
 }
-export declare function isHttpResponseRedirect(obj: any): boolean;
+export declare function isHttpResponseRedirect(obj: any): obj is HttpResponseRedirect;
 export declare abstract class HttpResponseClientError extends HttpResponse {
@@ -62,3 +62,3 @@ readonly isHttpResponseClientError: boolean;
 }
-export declare function isHttpResponseClientError(obj: any): boolean;
+export declare function isHttpResponseClientError(obj: any): obj is HttpResponseClientError;
 export declare class HttpResponseBadRequest extends HttpResponseClientError {
@@ -70,3 +70,3 @@ readonly isHttpResponseBadRequest: boolean;
 }
-export declare function isHttpResponseBadRequest(obj: any): boolean;
+export declare function isHttpResponseBadRequest(obj: any): obj is HttpResponseBadRequest;
 export declare class HttpResponseUnauthorized extends HttpResponseClientError {
@@ -81,3 +81,3 @@ readonly isHttpResponseUnauthorized: boolean;
 }
-export declare function isHttpResponseUnauthorized(obj: any): boolean;
+export declare function isHttpResponseUnauthorized(obj: any): obj is HttpResponseUnauthorized;
 export declare class HttpResponseForbidden extends HttpResponseClientError {
@@ -89,3 +89,3 @@ readonly isHttpResponseForbidden: boolean;
 }
-export declare function isHttpResponseForbidden(obj: any): boolean;
+export declare function isHttpResponseForbidden(obj: any): obj is HttpResponseForbidden;
 export declare class HttpResponseNotFound extends HttpResponseClientError {
@@ -97,3 +97,3 @@ readonly isHttpResponseNotFound: boolean;
 }
-export declare function isHttpResponseNotFound(obj: any): boolean;
+export declare function isHttpResponseNotFound(obj: any): obj is HttpResponseNotFound;
 export declare class HttpResponseMethodNotAllowed extends HttpResponseClientError {
@@ -105,3 +105,3 @@ readonly isHttpResponseMethodNotAllowed: boolean;
 }
-export declare function isHttpResponseMethodNotAllowed(obj: any): boolean;
+export declare function isHttpResponseMethodNotAllowed(obj: any): obj is HttpResponseMethodNotAllowed;
 export declare class HttpResponseConflict extends HttpResponseClientError {
@@ -113,3 +113,3 @@ readonly isHttpResponseConflict: boolean;
 }
-export declare function isHttpResponseConflict(obj: any): boolean;
+export declare function isHttpResponseConflict(obj: any): obj is HttpResponseConflict;
 export declare abstract class HttpResponseServerError extends HttpResponse {
@@ -119,3 +119,3 @@ readonly isHttpResponseServerError: boolean;
 }
-export declare function isHttpResponseServerError(obj: any): boolean;
+export declare function isHttpResponseServerError(obj: any): obj is HttpResponseServerError;
 export declare class HttpResponseInternalServerError extends HttpResponseServerError {
@@ -127,3 +127,3 @@ readonly isHttpResponseInternalServerError: boolean;
 }
-export declare function isHttpResponseInternalServerError(obj: any): boolean;
+export declare function isHttpResponseInternalServerError(obj: any): obj is HttpResponseInternalServerError;
 export declare class HttpResponseNotImplemented extends HttpResponseServerError {
@@ -135,2 +135,2 @@ readonly isHttpResponseNotImplemented: boolean;
 }
-export declare function isHttpResponseNotImplemented(obj: any): boolean;
+export declare function isHttpResponseNotImplemented(obj: any): obj is HttpResponseNotImplemented;

lib/core/index.d.ts

 export * from './class.interface';
-export { Controller } from './controllers';
+export { createController, Controller } from './controllers';
 export * from './http';
@@ -4,0 +4,0 @@ export * from './hooks';

lib/core/index.js

@@ -7,2 +7,3 @@ "use strict";
 var controllers_1 = require("./controllers");
+exports.createController = controllers_1.createController;
 exports.Controller = controllers_1.Controller;
@@ -9,0 +10,0 @@ __export(require("./http"));

lib/core/routes/make-controller-routes.js

 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+const controllers_1 = require("../controllers");
 const utils_1 = require("./utils");
@@ -14,4 +15,3 @@ function getMethods(obj) {
     const controllerPath = utils_1.getMetadata('path', controllerClass);
-    const controllerDependencies = utils_1.getMetadata('design:paramtypes', controllerClass) || [];
-    const controller = new controllerClass(...controllerDependencies.map(serviceClass => services.get(serviceClass)));
+    const controller = controllers_1.createController(controllerClass, services);
     getMethods(controllerClass.prototype).forEach(propertyKey => {
@@ -18,0 +18,0 @@ if (propertyKey === 'constructor') {

lib/express/create-app.d.ts

 import { Class, IModule } from '../core';
-export declare function createApp(rootModuleClass: Class<IModule>): any;
+export interface CreateAppOptions {
+    store?(session: any): any;
+}
+export declare function createApp(rootModuleClass: Class<IModule>, options?: CreateAppOptions): any;

lib/express/create-app.js

@@ -13,3 +13,3 @@ "use strict";
 const not_found_1 = require("./not-found");
-function createApp(rootModuleClass) {
+function createApp(rootModuleClass, options = {}) {
     const app = express();
@@ -20,2 +20,3 @@ app.use(logger('[:date] ":method :url HTTP/:http-version" :status - :response-time ms'));
     app.use(express.json());
+    app.use(express.urlencoded({ extended: false }));
     app.use(session({
@@ -33,2 +34,3 @@ cookie: {
         secret: core_1.Config.get('settings', 'sessionSecret', 'default_secret'),
+        store: options.store ? options.store(session) : undefined,
     }));
@@ -35,0 +37,0 @@ if (core_1.Config.get('settings', 'csrf', false)) {

lib/express/index.d.ts

@@ -1,1 +0,1 @@
-export { createApp } from './create-app';
+export { createApp, CreateAppOptions } from './create-app';

package.json

 {
   "name": "@foal/core",
-  "version": "0.6.0-alpha.3",
+  "version": "0.6.0-alpha.4",
   "description": "A high-level Node.JS framework to quickly build web apps in TypeScript.",
@@ -9,2 +9,4 @@ "main": "./lib/index.js",
     "test": "nyc --reporter=html --reporter=text mocha --require ts-node/register --require source-map-support/register \"./src/**/*.spec.ts\"",
+    "e2e": "mocha --require ts-node/register \"./e2e/**/*.spec.ts\"",
+    "dev:e2e": "mocha --require ts-node/register --watch --watch-extensions ts \"./e2e/**/*.spec.ts\"",
     "dev:test": "mocha --require ts-node/register --watch --watch-extensions ts \"./src/**/*.spec.ts\"",
@@ -80,2 +82,3 @@ "dev:test:auth": "mocha --require ts-node/register --watch --watch-extensions ts \"./src/auth/**/*.spec.ts\"",
     "@types/node": "^10.1.2",
+    "@types/supertest": "^2.0.5",
     "mocha": "^3.5.3",
@@ -82,0 +85,0 @@ "mysql": "^2.15.0",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

127074

increased by1.97%

Lines of code

1967

increased by1.44%

Worsened metrics

Dev dependency count

16

increased by6.67%

No dependency changes