Publish v0.1.1

- feat: custom responses for cache misses while network access is disabled
- feat: custom responses for unmocked requests while network access is disabled
- feat: `--reset`/`-r` reset flag to restart jambox
- fix: allow `localhost` requests even if network access is disabled #42

CHANGELOG.md

@@ -1,8 +1,9 @@
 ## 0.1.1 - `--reset` cli arg, better messages
 
-- Add custom responses for cache misses while network access is disabled
-- Add custom responses for unmocked requests while network access is disabled
-- Add `--reset`/`-r` reset flag to restart jambox
+- feat: custom responses for cache misses while network access is disabled
+- feat: custom responses for unmocked requests while network access is disabled
+- feat: `--reset`/`-r` reset flag to restart jambox
+- fix: allow `localhost` requests even if network access is disabled #42
 
 ## 0.1.0 - Refactor
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "jambox",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Tool for recording and playing back HTTP requests.",
   "bin": {
     "jam": "./jam.mjs",

src/Jambox.mjs

@@ -73,8 +73,24 @@ export default class Jambox extends Emitter {
     } else {
       await this.proxy
         .forAnyRequest()
+        .matching((req) => {
+          const url = new URL(req.url);
+          return url.hostname !== 'localhost';
+        })
         .asPriority(98)
         .thenReply(418, 'Network access disabled', '');
+
+      // Even if network access is disabled, allow localhost requests (for any port).
+      // Priority set to 1 in-case the user want's to override this behavior
+      // See https://github.com/ballercat/jambox/issues/42
+      await this.proxy
+        .forAnyRequest()
+        .matching((req) => {
+          const url = new URL(req.url);
+          return url.hostname === 'localhost';
+        })
+        .asPriority(1)
+        .thenPassThrough();
     }
 
     if (this.config.paused) {

src/__tests__/server.test.mjs

@@ -198,6 +198,34 @@ test.serial('pause', async (t) => {
   t.is(res.statusText, 'jambox stub');
 });
 
+test.serial('blocked network still allows localhost requests', async (t) => {
+  t.assert(t.context.server, `Server init error: ${t.context.error?.stack}`);
+
+  const { body: config } = await supertest(t.context.server).get('/api/config');
+
+  const opts = { agent: new HttpsProxyAgent(config.proxy.http) };
+
+  // works prior to blocking network requests
+  let response = await fetch(`http://localhost:${APP_PORT}/foobar`, opts);
+
+  t.is(response.status, 200);
+
+  let json = await response.json();
+  t.like(json, { path: '/foobar' });
+
+  await supertest(t.context.server)
+    .post('/api/config')
+    .send({ blockNetworkRequests: true })
+    .expect(200);
+
+  response = await fetch(`http://localhost:${APP_PORT}/foobar`, opts);
+
+  t.is(response.status, 200);
+
+  json = await response.json();
+  t.like(json, { path: '/foobar' });
+});
+
 // NOTE: This does work but needs a better cache mock
 test.serial('server - reset', async (t) => {
   t.assert(t.context.server, `Server init error: ${t.context.error?.stack}`);