insomnia-plugin-response - npm Package Compare versions

Comparing version 2.3.0-beta.2 to 2.3.0-beta.4

__tests__/index.test.js

@@ -543,9 +543,7 @@ const tag = require('..').templateTags[0];
 
-    it('does not resend recursive', async () => {
+    it('does not resend if request has already sent in recursive chain', async () => {
       const requests = [{ _id: 'req_1', parentId: 'wrk_1' }];
-
       const responses = [];
+      const context = _genTestContext(requests, responses, { requestChain: ['req_1']});
 
-      const context = _genTestContext(requests, responses, { fromResponseTag: true });
-
       try {
@@ -559,5 +557,15 @@ await tag.run(context, 'raw', 'req_1', '', 'always');
       throw new Error('Running tag should have thrown exception');
-    });
   });
 
+  it('does send if request has not been sent in recursive chain', async () => {
+    const requests = [{ _id: 'req_1', parentId: 'wrk_1' }];
+    const responses = [];
+
+    const context = _genTestContext(requests, responses, { requestChain: ['req_2']});
+
+    const response = await tag.run(context, 'raw', 'req_1', '', 'always');
+      expect(response).toBe('Response res_1')
+  });
+});
+
   describe('Max Age', () => {
@@ -564,0 +572,0 @@ const maxAgeArg = tag.args[4];

index.js

@@ -124,22 +124,30 @@ const jq = require('jsonpath');
       let shouldResend = false;
-      if (context.context.getExtraInfo('fromResponseTag')) {
-        shouldResend = false;
-      } else if (resendBehavior === 'never') {
-        shouldResend = false;
-      } else if (resendBehavior === 'no-history') {
-        shouldResend = !response;
-      } else if (resendBehavior === 'when-expired') {
-        if (!response) {
+      switch (resendBehavior) {
+        case 'no-history':
+          shouldResend = !response;
+          break;
+
+        case 'when-expired':
+          if (!response) {
+            shouldResend = true;
+          } else {
+            const ageSeconds = (Date.now() - response.created) / 1000;
+            shouldResend = ageSeconds > maxAgeSeconds;
+          }
+          break;
+        
+        case 'always':
           shouldResend = true;
-        } else {
-          const ageSeconds = (Date.now() - response.created) / 1000;
-          shouldResend = ageSeconds > maxAgeSeconds;
-        }
-      } else if (resendBehavior === 'always') {
-        shouldResend = true;
+          break;
+
+        case 'never':
+        default:
+          shouldResend = false;
+          break;
+
       }
 
       // Make sure we only send the request once per render so we don't have infinite recursion
-      const fromResponseTag = context.context.getExtraInfo('fromResponseTag');
-      if (fromResponseTag) {
+      const requestChain = context.context.getExtraInfo('requestChain') || [];
+      if (requestChain.some(id => id === request._id)) {
         console.log('[response tag] Preventing recursive render');
@@ -151,4 +159,5 @@ shouldResend = false;
         console.log('[response tag] Resending dependency');
+        requestChain.push(request._id)
         response = await context.network.sendRequest(request, [
-          { name: 'fromResponseTag', value: true },
+          { name: 'requestChain', value: requestChain }
         ]);
@@ -177,40 +186,39 @@ }
       const sanitizedFilter = filter.trim();
+      const bodyBuffer = context.util.models.response.getBodyBuffer(response, '');
+      const match = response.contentType && response.contentType.match(/charset=([\w-]+)/);
+      const charset = match && match.length >= 2 ? match[1] : 'utf-8';
+      switch (field) {
+        case 'header':
+          return matchHeader(response.headers, sanitizedFilter);
 
-      if (field === 'header') {
-        return matchHeader(response.headers, sanitizedFilter);
-      } else if (field === 'url') {
-        return response.url;
-      } else if (field === 'raw') {
-        const bodyBuffer = context.util.models.response.getBodyBuffer(response, '');
-        const match = response.contentType.match(/charset=([\w-]+)/);
-        const charset = match && match.length >= 2 ? match[1] : 'utf-8';
+        case 'url':
+          return response.url;
 
-        // Sometimes iconv conversion fails so fallback to regular buffer
-        try {
-          return iconv.decode(bodyBuffer, charset);
-        } catch (err) {
-          console.warn('[response] Failed to decode body', err);
-          return bodyBuffer.toString();
-        }
-      } else if (field === 'body') {
-        const bodyBuffer = context.util.models.response.getBodyBuffer(response, '');
-        const match = response.contentType.match(/charset=([\w-]+)/);
-        const charset = match && match.length >= 2 ? match[1] : 'utf-8';
+        case 'raw':
+          // Sometimes iconv conversion fails so fallback to regular buffer
+          try {
+            return iconv.decode(bodyBuffer, charset);
+          } catch (err) {
+            console.warn('[response] Failed to decode body', err);
+            return bodyBuffer.toString();
+          }
 
-        // Sometimes iconv conversion fails so fallback to regular buffer
-        let body;
-        try {
-          body = iconv.decode(bodyBuffer, charset);
-        } catch (err) {
-          body = bodyBuffer.toString();
-          console.warn('[response] Failed to decode body', err);
-        }
+        case 'body':
+          // Sometimes iconv conversion fails so fallback to regular buffer
+          let body;
+          try {
+            body = iconv.decode(bodyBuffer, charset);
+          } catch (err) {
+            body = bodyBuffer.toString();
+            console.warn('[response] Failed to decode body', err);
+          }
+    
+          if (sanitizedFilter.indexOf('$') === 0) {
+            return matchJSONPath(body, sanitizedFilter);
+          } else {
+            return matchXPath(body, sanitizedFilter);
+          }
 
-        if (sanitizedFilter.indexOf('$') === 0) {
-          return matchJSONPath(body, sanitizedFilter);
-        } else {
-          return matchXPath(body, sanitizedFilter);
-        }
-      } else {
-        throw new Error(`Unknown field ${field}`);
+        default:
+          throw new Error(`Unknown field ${field}`);
       }
@@ -217,0 +225,0 @@ },

package.json

 {
   "name": "insomnia-plugin-response",
-  "version": "2.3.0-beta.2",
+  "version": "2.3.0-beta.4",
   "author": "Kong <[e-mail]>",
@@ -29,3 +29,3 @@ "description": "Insomnia response template tag",
   },
-  "gitHead": "34b8c1726019b2e87c5df678086c7896b3bc6389"
+  "gitHead": "af8e59b4ea981ca11c219521b658da5ed7e02c21"
 }

No alert changes

Improved metrics

Total package byte prevSize

30108

increased by0.84%

Lines of code

818

increased by1.24%

No dependency changes