refactor(http-client): inherit interceptors from parent instance

packages/backend/src/apps/ntfy/actions/send-message/index.js

@@ -89,8 +89,6 @@ export default defineAction({
 
     const response = await $.http.post('/', payload);
 
-    console.log(response.config.additionalProperties.extraData);
-
     $.setActionItem({
       raw: response.data,
     });

packages/backend/src/apps/ntfy/common/add-auth-header.js

@@ -1,7 +1,4 @@
 const addAuthHeader = ($, requestConfig) => {
-  console.log('requestConfig', requestConfig)
-  if (requestConfig.additionalProperties?.skip) return requestConfig;
-
   if ($.auth.data.serverUrl) {
     requestConfig.baseURL = $.auth.data.serverUrl;
   }

packages/backend/src/apps/ntfy/common/async-before-request.js

@@ -1,23 +0,0 @@
-const asyncBeforeRequest = async ($, requestConfig) => {
-  if (requestConfig.additionalProperties?.skip)
-    return requestConfig;
-
-  const response = await $.http.post(
-    'http://localhost:3000/webhooks/flows/8a040f4e-817f-4076-80ba-3c1c0af7e65e/sync',
-    null,
-    {
-      additionalProperties: {
-        skip: true,
-      },
-    }
-  );
-
-  console.log(response);
-  requestConfig.additionalProperties = {
-    extraData: response.data
-  }
-
-  return requestConfig;
-};
-
-export default asyncBeforeRequest;

packages/backend/src/apps/ntfy/index.js

@@ -1,6 +1,5 @@
 import defineApp from '../../helpers/define-app.js';
 import addAuthHeader from './common/add-auth-header.js';
-import asyncBeforeRequest from './common/async-before-request.js';
 import auth from './auth/index.js';
 import actions from './actions/index.js';
 
@@ -13,7 +12,7 @@ export default defineApp({
   baseUrl: 'https://ntfy.sh',
   apiBaseUrl: 'https://ntfy.sh',
   primaryColor: '56bda8',
-  beforeRequest: [asyncBeforeRequest, addAuthHeader],
+  beforeRequest: [addAuthHeader],
   auth,
   actions,
 });

packages/backend/src/helpers/axios-with-proxy.js

@@ -3,18 +3,14 @@ import { HttpsProxyAgent } from 'https-proxy-agent';
 import { HttpProxyAgent } from 'http-proxy-agent';
 import appConfig from '../config/app.js';
 
-export function createInstance(customConfig = {}, { requestInterceptor, responseErrorInterceptor } = {}) {
+const config = axios.defaults;
-  const config = {
+const httpProxyUrl = appConfig.httpProxy;
-    ...axios.defaults,
+const httpsProxyUrl = appConfig.httpsProxy;
-    ...customConfig
+const supportsProxy = httpProxyUrl || httpsProxyUrl;
-  };
+const noProxyEnv = appConfig.noProxy;
-  const httpProxyUrl = appConfig.httpProxy;
+const noProxyHosts = noProxyEnv ? noProxyEnv.split(',').map(host => host.trim()) : [];
-  const httpsProxyUrl = appConfig.httpsProxy;
-  const supportsProxy = httpProxyUrl || httpsProxyUrl;
-  const noProxyEnv = appConfig.noProxy;
-  const noProxyHosts = noProxyEnv ? noProxyEnv.split(',').map(host => host.trim()) : [];
 
-  if (supportsProxy) {
+if (supportsProxy) {
   if (httpProxyUrl) {
     config.httpAgent = new HttpProxyAgent(httpProxyUrl);
   }
@@ -24,20 +20,20 @@ export function createInstance(customConfig = {}, { requestInterceptor, response
   }
 
   config.proxy = false;
-  }
+}
 
-  const instance = axios.create(config);
+const axiosWithProxyInstance = axios.create(config);
 
-  function shouldSkipProxy(hostname) {
+function shouldSkipProxy(hostname) {
   return noProxyHosts.some(noProxyHost => {
     return hostname.endsWith(noProxyHost) || hostname === noProxyHost;
   });
-  };
+};
 
-  /**
+/**
  * The interceptors are executed in the reverse order they are added.
  */
-  instance.interceptors.request.use(
+axiosWithProxyInstance.interceptors.request.use(
   function skipProxyIfInNoProxy(requestConfig) {
     const hostname = new URL(requestConfig.baseURL).hostname;
 
@@ -48,25 +44,11 @@ export function createInstance(customConfig = {}, { requestInterceptor, response
 
     return requestConfig;
   },
-    (error) => Promise.reject(error)
+  undefined,
-  );
+  { synchronous: true }
+);
 
-  // not always we have custom request interceptors
+axiosWithProxyInstance.interceptors.request.use(
-  if (requestInterceptor) {
-    instance.interceptors.request.use(
-      async function customInterceptor(requestConfig) {
-        let newRequestConfig = requestConfig;
-
-        for (const interceptor of requestInterceptor) {
-          newRequestConfig = await interceptor(newRequestConfig);
-        }
-
-        return newRequestConfig;
-      }
-    );
-  }
-
-  instance.interceptors.request.use(
   function removeBaseUrlForAbsoluteUrls(requestConfig) {
     /**
      * If the URL is an absolute URL, we remove its origin out of the URL
@@ -79,24 +61,12 @@ export function createInstance(customConfig = {}, { requestInterceptor, response
       requestConfig.url = url.pathname + url.search;
 
       return requestConfig;
-      } catch (err) {
+    } catch {
       return requestConfig;
     }
   },
-    (error) => Promise.reject(error)
+  undefined,
-  );
+  { synchronous: true}
+);
 
-  // not always we have custom response error interceptor
+export default axiosWithProxyInstance;
-  if (responseErrorInterceptor) {
-    instance.interceptors.response.use(
-      (response) => response,
-      responseErrorInterceptor
-    );
-  }
-
-  return instance;
-}
-
-const defaultInstance = createInstance();
-
-export default defaultInstance;

packages/backend/src/helpers/axios-with-proxy.test.js

@@ -1,6 +1,6 @@
 import { beforeEach, describe, it, expect, vi } from 'vitest';
 
-describe('Custom default axios with proxy', () => {
+describe('Axios with proxy', () => {
   beforeEach(() => {
     vi.resetModules();
   });
@@ -23,13 +23,7 @@ describe('Custom default axios with proxy', () => {
     expect(secondRequestInterceptor.fulfilled.name).toBe('removeBaseUrlForAbsoluteUrls');
   });
 
-  it('should throw with invalid url (consisting of path alone)', async () => {
+  describe('skipProxyIfInNoProxy', () => {
-    const axios = (await import('./axios-with-proxy.js')).default;
-
-    await expect(() => axios('/just-a-path')).rejects.toThrowError('Invalid URL');
-  });
-
-  describe('with skipProxyIfInNoProxy interceptor', () => {
     let appConfig, axios;
     beforeEach(async() => {
       appConfig = (await import('../config/app.js')).default;
@@ -73,7 +67,7 @@ describe('Custom default axios with proxy', () => {
     });
   });
 
-  describe('with removeBaseUrlForAbsoluteUrls interceptor', () => {
+  describe('removeBaseUrlForAbsoluteUrls', () => {
     let axios;
     beforeEach(async() => {
       axios = (await import('./axios-with-proxy.js')).default;
@@ -122,48 +116,4 @@ describe('Custom default axios with proxy', () => {
       expect(interceptedRequestConfig.url).toBe('/path?query=1');
     });
   });
-
-  describe('with extra requestInterceptors', () => {
-    it('should apply extra request interceptors in the middle', async () => {
-      const { createInstance } = await import('./axios-with-proxy.js');
-
-      const interceptor = (config) => {
-        config.test = true;
-        return config;
-      }
-
-      const instance = createInstance({}, {
-        requestInterceptor: [
-          interceptor
-        ]
-      });
-      const requestInterceptors = instance.interceptors.request.handlers;
-      const customInterceptor = requestInterceptors[1].fulfilled;
-
-      expect(requestInterceptors.length).toBe(3);
-      await expect(customInterceptor({})).resolves.toStrictEqual({ test: true });
-    });
-
-    it('should work with a custom interceptor setting a baseURL and a request to path', async () => {
-      const { createInstance } = await import('./axios-with-proxy.js');
-
-      const interceptor = (config) => {
-        config.baseURL = 'http://localhost';
-        return config;
-      }
-
-      const instance = createInstance({}, {
-        requestInterceptor: [
-          interceptor
-        ]
-      });
-
-      try {
-        await instance.get('/just-a-path');
-      } catch (error) {
-        expect(error.config.baseURL).toBe('http://localhost');
-        expect(error.config.url).toBe('/just-a-path');
-      }
-    })
-  });
 });

packages/backend/src/helpers/http-client/index.js

@@ -1,8 +1,41 @@
 import HttpError from '../../errors/http.js';
-import { createInstance } from '../axios-with-proxy.js';
+import axios from '../axios-with-proxy.js';
+
+// Mutates the `toInstance` by copying the request interceptors from `fromInstance`
+const copyRequestInterceptors = (fromInstance, toInstance) => {
+  // Copy request interceptors
+  fromInstance.interceptors.request.forEach(interceptor => {
+    toInstance.interceptors.request.use(
+      interceptor.fulfilled,
+      interceptor.rejected,
+      {
+        synchronous: interceptor.synchronous,
+        runWhen: interceptor.runWhen
+      }
+    );
+  });
+}
 
 export default function createHttpClient({ $, baseURL, beforeRequest = [] }) {
-  async function interceptResponseError(error) {
+  const instance = axios.create({
+    baseURL,
+  });
+
+  // 1. apply the beforeRequest functions from the app
+  instance.interceptors.request.use((requestConfig) => {
+    const result = beforeRequest.reduce((newConfig, beforeRequestFunc) => {
+      return beforeRequestFunc($, newConfig);
+    }, requestConfig);
+
+    return result;
+  });
+
+  // 2. inherit the request inceptors from the parent instance
+  copyRequestInterceptors(axios, instance);
+
+  instance.interceptors.response.use(
+    (response) => response,
+    async (error) => {
       const { config, response } = error;
       // Do not destructure `status` from `error.response` because it might not exist
       const status = response?.status;
@@ -25,19 +58,8 @@ export default function createHttpClient({ $, baseURL, beforeRequest = [] }) {
       }
 
       throw new HttpError(error);
-  };
-
-  const instance = createInstance(
-    {
-      baseURL,
-    },
-    {
-      requestInterceptor: beforeRequest.map((originalBeforeRequest) => {
-        return async (requestConfig) => await originalBeforeRequest($, requestConfig);
-      }),
-      responseErrorInterceptor: interceptResponseError,
     }
-  )
+  );
 
   return instance;
 }

packages/backend/src/helpers/webhook-handler-sync.js

@@ -63,8 +63,6 @@ export default async (flowId, request, response) => {
     });
 
     if (testRun) {
-      response.status(204).end();
-
       // in case of testing, we do not process the whole process.
       continue;
     }
@@ -76,12 +74,6 @@ export default async (flowId, request, response) => {
         executionId,
       });
 
-      if (actionStep.appKey === 'filter' && !actionExecutionStep.dataOut) {
-        response.status(422).end();
-
-        break;
-      }
-
       if (actionStep.key === 'respondWith' && !response.headersSent) {
         const { headers, statusCode, body } = actionExecutionStep.dataOut;
 

packages/docs/pages/apps/appwrite/connection.md

@@ -14,7 +14,7 @@ connection in Automatisch. If any of the steps are outdated, please let us know!
 7. Click on the **Select all** and then click on the **Create** button.
 8. Now, copy your **API key secret** and paste the key into the **API Key** field in Automatisch.
 9. Write any screen name to be displayed in Automatisch.
-10. You can find your project ID next to your project name. Paste the id into **Project ID** field in Automatisch.
+10. You can find your project ID next to your project name. Paste the id into **Project ID** field in Automatsich.
-11. If you are using self-hosted Appwrite project, you can paste the instance url into **Appwrite instance URL** field in Automatisch.
+11. If you are using self-hosted Appwrite project, you can paste the instace url into **Appwrite instance URL** field in Automatisch.
 12. Fill the host name field with the hostname of your instance URL. It's either `cloud.appwrite.io` or hostname of your instance URL.
 13. Start using Appwrite integration with Automatisch!

packages/docs/pages/apps/appwrite/triggers.md

@@ -1,7 +1,7 @@
 ---
 favicon: /favicons/appwrite.svg
 items:
-  - name: New documents
+  - name: New documets
     desc: Triggers when a new document is created.
 ---