mvcjs - npm Package Compare versions

Comparing version 0.1.17 to 0.1.18

framework/core/request.js

@@ -381,2 +381,4 @@ "use strict";
 
+            this.isRendered = true;
+
             logger.info('Request.forward.url:', {
@@ -386,7 +388,2 @@ url: url
 
-            // destroy current request on end of request
-            request.onEnd(function () {
-                that._destroy();
-            });
-
             return request.parse();
@@ -425,2 +422,4 @@ }
 
+            this.isRendered = true;
+
             logger.info('Request.forward.route:', {
@@ -469,3 +468,3 @@ route: route,
         this.setStatusCode(304);
-        this._render('NO CHANGE');
+        this._render('MVCJS_NO_STATUS_CHANGE');
     },
@@ -551,3 +550,2 @@ /**
                     .then(this._resolveRoute.bind(this)); // resolve route chain
-
             }.bind(this)) // handle hook chain
@@ -578,3 +576,8 @@ .then(this._compress.bind(this))
 
+        if (this.isRendered) {
+            // we have multiple recursion in parse for catching
+            return false;
+        }
 
+
         if (isForCompress && !!this.isCompressionEnabled && Type.isString(accept)) {
@@ -665,4 +668,5 @@ if (!this.hasHeader('Vary')) {
         if (!this.isERROR && !!router.getErrorRoute()) {
+            // return new request
+            this.isRendered = true;
 
-            // return new request
             request = new Request({
@@ -683,7 +687,6 @@ request: this.request,
             // return parsed request
-
             return request.parse();
         } else {
             this.addHeader('Content-Type', 'text/plain');
-            return this._render(response);
+            return response;
         }
@@ -690,0 +693,0 @@ },

package.json

@@ -5,3 +5,3 @@ {
   "description": "Powerful lightweight mvc framework for nodejs",
-  "version": "0.1.17",
+  "version": "0.1.18",
   "dependencies": {
@@ -8,0 +8,0 @@ "mongoose": "4.1.x",

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

160482

increased by0.08%

Lines of code

5138

increased by0.06%

No dependency changes