@unito/integration-sdk 0.1.5 → 0.1.7

package/dist/src/handler.js

@@ -16,7 +16,7 @@ function assertValidConfiguration(path, pathWithIdentifier, handlers) {
         const hasIndividualHandlers = individualHandlers.some(handler => handler in handlers);
         const hasCollectionHandlers = collectionHandlers.some(handler => handler in handlers);
         if (hasIndividualHandlers && hasCollectionHandlers) {
-            throw new InvalidHandler(`The provided path '${path}' doesn't differentiate between individual and collection level operation, so you cannot define both. `);
+            throw new InvalidHandler(`The provided path '${path}' doesn't differentiate between individual and collection level operation, so you cannot define both.`);
         }
     }
 }
@@ -83,7 +83,7 @@ export class Handler {
         console.debug(`\x1b[33mMounting handler at path ${this.pathWithIdentifier}`);
         if (this.handlers.getCollection) {
             const handler = this.handlers.getCollection;
-            console.debug(`  Enabling GET ${this.path}`);
+            console.debug(`  Enabling getCollection at GET ${this.path}`);
             router.get(this.path, async (req, res) => {
                 if (!res.locals.credentials) {
                     throw new UnauthorizedError();
@@ -101,7 +101,7 @@ export class Handler {
         }
         if (this.handlers.createItem) {
             const handler = this.handlers.createItem;
-            console.debug(`  Enabling POST ${this.path}`);
+            console.debug(`  Enabling createItem at POST ${this.path}`);
             router.post(this.path, async (req, res) => {
                 if (!res.locals.credentials) {
                     throw new UnauthorizedError();
@@ -119,7 +119,7 @@ export class Handler {
         }
         if (this.handlers.getItem) {
             const handler = this.handlers.getItem;
-            console.debug(`  Enabling GET ${this.pathWithIdentifier}`);
+            console.debug(`  Enabling getItem at GET ${this.pathWithIdentifier}`);
             router.get(this.pathWithIdentifier, async (req, res) => {
                 if (!res.locals.credentials) {
                     throw new UnauthorizedError();
@@ -135,7 +135,7 @@ export class Handler {
         }
         if (this.handlers.updateItem) {
             const handler = this.handlers.updateItem;
-            console.debug(`  Enabling PATCH ${this.pathWithIdentifier}`);
+            console.debug(`  Enabling updateItem at PATCH ${this.pathWithIdentifier}`);
             router.patch(this.pathWithIdentifier, async (req, res) => {
                 if (!res.locals.credentials) {
                     throw new UnauthorizedError();
@@ -153,7 +153,7 @@ export class Handler {
         }
         if (this.handlers.deleteItem) {
             const handler = this.handlers.deleteItem;
-            console.debug(`  Enabling DELETE ${this.pathWithIdentifier}`);
+            console.debug(`  Enabling deleteItem at DELETE ${this.pathWithIdentifier}`);
             router.delete(this.pathWithIdentifier, async (req, res) => {
                 if (!res.locals.credentials) {
                     throw new UnauthorizedError();
@@ -169,7 +169,7 @@ export class Handler {
         }
         if (this.handlers.getCredentialAccount) {
             const handler = this.handlers.getCredentialAccount;
-            console.debug(`  Enabling GET ${this.pathWithIdentifier}`);
+            console.debug(`  Enabling getCredentialAccount at GET ${this.pathWithIdentifier}`);
             router.get(this.pathWithIdentifier, async (req, res) => {
                 if (!res.locals.credentials) {
                     throw new UnauthorizedError();
@@ -185,7 +185,7 @@ export class Handler {
         }
         if (this.handlers.acknowledgeWebhooks) {
             const handler = this.handlers.acknowledgeWebhooks;
-            console.debug(`  Enabling POST ${this.pathWithIdentifier}`);
+            console.debug(`  Enabling acknowledgeWebhooks at POST ${this.pathWithIdentifier}`);
             router.post(this.pathWithIdentifier, async (req, res) => {
                 assertWebhookParseRequestPayload(req.body);
                 const response = await handler({
@@ -199,7 +199,7 @@ export class Handler {
         }
         if (this.handlers.parseWebhooks) {
             const handler = this.handlers.parseWebhooks;
-            console.debug(`  Enabling POST ${this.pathWithIdentifier}`);
+            console.debug(`  Enabling parseWebhooks at POST ${this.pathWithIdentifier}`);
             router.post(this.pathWithIdentifier, async (req, res) => {
                 assertWebhookParseRequestPayload(req.body);
                 const response = await handler({
@@ -213,7 +213,7 @@ export class Handler {
         }
         if (this.handlers.updateWebhookSubscriptions) {
             const handler = this.handlers.updateWebhookSubscriptions;
-            console.debug(`  Enabling PUT ${this.pathWithIdentifier}`);
+            console.debug(`  Enabling updateWebhookSubscriptions at PUT ${this.pathWithIdentifier}`);
             router.put(this.pathWithIdentifier, async (req, res) => {
                 if (!res.locals.credentials) {
                     throw new UnauthorizedError();

package/dist/src/integration.d.ts

@@ -1,9 +1,14 @@
 import { HandlersInput } from './handler.js';
+type Options = {
+    port?: number;
+};
 export default class Integration {
     private handlers;
     private instance;
     private cache;
-    constructor();
+    private port;
+    constructor(options?: Options);
     addHandler(path: string, handlers: HandlersInput): void;
     start(): void;
 }
+export {};

package/dist/src/integration.js

@@ -18,7 +18,9 @@ export default class Integration {
     handlers;
     instance = undefined;
     cache = undefined;
-    constructor() {
+    port;
+    constructor(options = {}) {
+        this.port = options.port || 9200;
         this.handlers = [];
     }
     addHandler(path, handlers) {
@@ -85,7 +87,8 @@ export default class Integration {
         app.use(errorsMiddleware);
         // Must be the last handler.
         app.use(notFoundMiddleware);
-        this.instance = app.listen(process.env.PORT || 9200, () => console.info(`Server started on port ${process.env.PORT || 9200}.`));
+        // Start the server.
+        this.instance = app.listen(this.port, () => console.info(`Server started on port ${this.port}.`));
         // Trap exit signals.
         ['SIGTERM', 'SIGINT', 'SIGUSR2'].forEach(signalType => {
             process.once(signalType, async () => {

package/dist/src/resources/provider.d.ts

@@ -1,4 +1,5 @@
 import { Credentials } from '../middlewares/credentials.js';
+import Logger from '../resources/logger.js';
 /**
  * RateLimiter is a wrapper function that you can provide to limit the rate of calls to the provider based on the
  * caller's credentials.
@@ -17,9 +18,11 @@ import { Credentials } from '../middlewares/credentials.js';
  */
 export type RateLimiter = <T>(context: {
     credentials: Credentials;
+    logger: Logger;
 }, targetFunction: () => Promise<Response<T>>) => Promise<Response<T>>;
 export interface RequestOptions {
     credentials: Credentials;
+    logger: Logger;
     queryParams?: {
         [key: string]: string;
     };
@@ -28,7 +31,7 @@ export interface RequestOptions {
     };
 }
 export interface Response<T> {
-    data: T;
+    body: T;
     status: number;
     headers: Headers;
 }
@@ -36,6 +39,7 @@ export declare class Provider {
     protected rateLimiter: RateLimiter | undefined;
     protected prepareRequest: (context: {
         credentials: Credentials;
+        logger: Logger;
     }) => {
         url: string;
         headers: Record<string, string>;

package/dist/src/resources/provider.js

@@ -63,7 +63,7 @@ export class Provider {
         });
     }
     async fetchWrapper(endpoint, body, options) {
-        const { url: providerUrl, headers: providerHeaders } = this.prepareRequest({ credentials: options.credentials });
+        const { url: providerUrl, headers: providerHeaders } = this.prepareRequest(options);
         let absoluteUrl = [providerUrl, endpoint.charAt(0) === '/' ? endpoint.substring(1) : endpoint].join('/');
         if (options.queryParams) {
             absoluteUrl = `${absoluteUrl}?${new URLSearchParams(options.queryParams)}`;
@@ -89,8 +89,8 @@ export class Provider {
                 throw buildHttpError(response.status, textResult);
             }
             try {
-                const data = response.body ? await response.json() : undefined;
-                return { status: response.status, headers: response.headers, data };
+                const body = response.body ? await response.json() : undefined;
+                return { status: response.status, headers: response.headers, body };
             }
             catch {
                 throw buildHttpError(400, 'Invalid JSON response');

package/dist/test/integration.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/test/integration.test.js

@@ -0,0 +1,26 @@
+import assert from 'node:assert/strict';
+import { describe, it } from 'node:test';
+import Integration from '../src/integration.js';
+describe('Integration', () => {
+    describe('constructor', () => {
+        it('defaults to port 9200', () => {
+            const integration = new Integration();
+            assert.equal(integration['port'], 9200);
+        });
+        it('accepts an optional port', () => {
+            const integration = new Integration({ port: 1234 });
+            assert.equal(integration['port'], 1234);
+        });
+    });
+    describe('addHandler', () => {
+        it('works', () => {
+            const integration = new Integration();
+            integration.addHandler('/', {});
+            const handler = integration['handlers'][0];
+            assert.ok(handler);
+            assert.equal(handler['path'], '/');
+            assert.equal(handler['pathWithIdentifier'], '/');
+            assert.deepEqual(handler['handlers'], {});
+        });
+    });
+});

package/dist/test/middlewares/filters.test.js

@@ -15,6 +15,14 @@ describe('filters middleware', () => {
             ],
         });
     });
+    it('decodes URI components', () => {
+        const request = { query: { filter: 'status=foo%2Cbar!!%2C%3Fbaz%3D!%3Equx' } };
+        const response = { locals: {} };
+        middleware(request, response, () => { });
+        assert.deepEqual(response.locals, {
+            filters: [{ field: 'status', operator: OperatorType.EQUAL, values: ['foo,bar!!,?baz=!>qux'] }],
+        });
+    });
     it('no data', () => {
         const request = { query: {} };
         const response = { locals: {} };

package/dist/test/resources/provider.test.js

@@ -1,6 +1,7 @@
 import assert from 'node:assert/strict';
 import { describe, it } from 'node:test';
 import { Provider } from '../../src/resources/provider.js';
+import Logger from '../../src/resources/logger.js';
 describe('Provider', () => {
     const provider = new Provider({
         prepareRequest: requestOptions => {
@@ -13,6 +14,7 @@ describe('Provider', () => {
             };
         },
     });
+    const logger = new Logger();
     it('get', async (context) => {
         const response = new Response('{"data": "value"}', {
             status: 200,
@@ -21,6 +23,7 @@ describe('Provider', () => {
         const fetchMock = context.mock.method(global, 'fetch', () => Promise.resolve(response));
         const actualResponse = await provider.get('/endpoint', {
             credentials: { apiKey: 'apikey#1111' },
+            logger: logger,
             additionnalheaders: { 'X-Additional-Header': 'value1' },
         });
         assert.equal(fetchMock.mock.calls.length, 1);
@@ -38,7 +41,7 @@ describe('Provider', () => {
                 },
             },
         ]);
-        assert.deepEqual(actualResponse, { status: 200, headers: response.headers, data: { data: 'value' } });
+        assert.deepEqual(actualResponse, { status: 200, headers: response.headers, body: { data: 'value' } });
     });
     it('post with url encoded body', async (context) => {
         const response = new Response('{"data": "value"}', {
@@ -50,6 +53,7 @@ describe('Provider', () => {
             data: 'createdItemInfo',
         }, {
             credentials: { apiKey: 'apikey#1111' },
+            logger: logger,
             additionnalheaders: { 'X-Additional-Header': 'value1' },
         });
         assert.equal(fetchMock.mock.calls.length, 1);
@@ -67,7 +71,7 @@ describe('Provider', () => {
                 },
             },
         ]);
-        assert.deepEqual(actualResponse, { status: 201, headers: response.headers, data: { data: 'value' } });
+        assert.deepEqual(actualResponse, { status: 201, headers: response.headers, body: { data: 'value' } });
     });
     it('put with json body', async (context) => {
         const response = new Response('{"data": "value"}', {
@@ -80,6 +84,7 @@ describe('Provider', () => {
             data: 'updatedItemInfo',
         }, {
             credentials: { apiKey: 'apikey#1111' },
+            logger: logger,
             additionnalheaders: { 'X-Additional-Header': 'value1', 'Content-Type': 'application/json' },
         });
         assert.equal(fetchMock.mock.calls.length, 1);
@@ -97,7 +102,7 @@ describe('Provider', () => {
                 },
             },
         ]);
-        assert.deepEqual(actualResponse, { status: 201, headers: response.headers, data: { data: 'value' } });
+        assert.deepEqual(actualResponse, { status: 201, headers: response.headers, body: { data: 'value' } });
     });
     it('patch with query params', async (context) => {
         const response = new Response('{"data": "value"}', {
@@ -109,6 +114,7 @@ describe('Provider', () => {
             data: 'updatedItemInfo',
         }, {
             credentials: { apiKey: 'apikey#1111' },
+            logger: logger,
             queryParams: { param1: 'value1', param2: 'value2' },
             additionnalheaders: { 'X-Additional-Header': 'value1', 'Content-Type': 'application/json' },
         });
@@ -127,7 +133,7 @@ describe('Provider', () => {
                 },
             },
         ]);
-        assert.deepEqual(actualResponse, { status: 201, headers: response.headers, data: { data: 'value' } });
+        assert.deepEqual(actualResponse, { status: 201, headers: response.headers, body: { data: 'value' } });
     });
     it('delete', async (context) => {
         const response = new Response(undefined, {
@@ -137,6 +143,7 @@ describe('Provider', () => {
         const fetchMock = context.mock.method(global, 'fetch', () => Promise.resolve(response));
         const actualResponse = await provider.delete('/endpoint/123', {
             credentials: { apiKey: 'apikey#1111' },
+            logger: logger,
             additionnalheaders: { 'X-Additional-Header': 'value1', 'Content-Type': 'application/json' },
         });
         assert.equal(fetchMock.mock.calls.length, 1);
@@ -154,7 +161,7 @@ describe('Provider', () => {
                 },
             },
         ]);
-        assert.deepEqual(actualResponse, { status: 204, headers: response.headers, data: undefined });
+        assert.deepEqual(actualResponse, { status: 204, headers: response.headers, body: undefined });
     });
     it('uses rate limiter if provided', async (context) => {
         const mockRateLimiter = context.mock.fn((_context, request) => Promise.resolve(request()));
@@ -177,6 +184,7 @@ describe('Provider', () => {
         const fetchMock = context.mock.method(global, 'fetch', () => Promise.resolve(response));
         const options = {
             credentials: { apiKey: 'apikey#1111' },
+            logger: logger,
             additionnalheaders: { 'X-Additional-Header': 'value1', 'Content-Type': 'application/json' },
         };
         const actualResponse = await rateLimitedProvider.delete('/endpoint/123', options);
@@ -197,7 +205,7 @@ describe('Provider', () => {
                 },
             },
         ]);
-        assert.deepEqual(actualResponse, { status: 204, headers: response.headers, data: undefined });
+        assert.deepEqual(actualResponse, { status: 204, headers: response.headers, body: undefined });
     });
     it('throws on invalid json response', async (context) => {
         const response = new Response('{invalidJSON}', {
@@ -206,6 +214,7 @@ describe('Provider', () => {
         context.mock.method(global, 'fetch', () => Promise.resolve(response));
         assert.rejects(() => provider.get('/endpoint/123', {
             credentials: { apiKey: 'apikey#1111' },
+            logger: logger,
         }));
     });
     it('throws on status 400', async (context) => {
@@ -215,6 +224,7 @@ describe('Provider', () => {
         context.mock.method(global, 'fetch', () => Promise.resolve(response));
         assert.rejects(() => provider.get('/endpoint/123', {
             credentials: { apiKey: 'apikey#1111' },
+            logger: logger,
         }));
     });
     it('throws on status 429', async (context) => {
@@ -224,6 +234,7 @@ describe('Provider', () => {
         context.mock.method(global, 'fetch', () => Promise.resolve(response));
         assert.rejects(() => provider.get('/endpoint/123', {
             credentials: { apiKey: 'apikey#1111' },
+            logger: logger,
         }));
     });
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@unito/integration-sdk",
-  "version": "0.1.5",
+  "version": "0.1.7",
   "description": "Integration SDK",
   "type": "module",
   "types": "dist/src/index.d.ts",

package/src/handler.ts

@@ -128,7 +128,7 @@ function assertValidConfiguration(path: Path, pathWithIdentifier: Path, handlers
 
     if (hasIndividualHandlers && hasCollectionHandlers) {
       throw new InvalidHandler(
-        `The provided path '${path}' doesn't differentiate between individual and collection level operation, so you cannot define both. `,
+        `The provided path '${path}' doesn't differentiate between individual and collection level operation, so you cannot define both.`,
       );
     }
   }
@@ -221,7 +221,7 @@ export class Handler {
     if (this.handlers.getCollection) {
       const handler = this.handlers.getCollection;
 
-      console.debug(`  Enabling GET ${this.path}`);
+      console.debug(`  Enabling getCollection at GET ${this.path}`);
 
       router.get(this.path, async (req, res) => {
         if (!res.locals.credentials) {
@@ -244,7 +244,7 @@ export class Handler {
     if (this.handlers.createItem) {
       const handler = this.handlers.createItem;
 
-      console.debug(`  Enabling POST ${this.path}`);
+      console.debug(`  Enabling createItem at POST ${this.path}`);
 
       router.post(this.path, async (req, res) => {
         if (!res.locals.credentials) {
@@ -268,7 +268,7 @@ export class Handler {
     if (this.handlers.getItem) {
       const handler = this.handlers.getItem;
 
-      console.debug(`  Enabling GET ${this.pathWithIdentifier}`);
+      console.debug(`  Enabling getItem at GET ${this.pathWithIdentifier}`);
 
       router.get(this.pathWithIdentifier, async (req, res) => {
         if (!res.locals.credentials) {
@@ -289,7 +289,7 @@ export class Handler {
     if (this.handlers.updateItem) {
       const handler = this.handlers.updateItem;
 
-      console.debug(`  Enabling PATCH ${this.pathWithIdentifier}`);
+      console.debug(`  Enabling updateItem at PATCH ${this.pathWithIdentifier}`);
 
       router.patch(this.pathWithIdentifier, async (req, res) => {
         if (!res.locals.credentials) {
@@ -313,7 +313,7 @@ export class Handler {
     if (this.handlers.deleteItem) {
       const handler = this.handlers.deleteItem;
 
-      console.debug(`  Enabling DELETE ${this.pathWithIdentifier}`);
+      console.debug(`  Enabling deleteItem at DELETE ${this.pathWithIdentifier}`);
 
       router.delete(this.pathWithIdentifier, async (req, res) => {
         if (!res.locals.credentials) {
@@ -334,7 +334,7 @@ export class Handler {
     if (this.handlers.getCredentialAccount) {
       const handler = this.handlers.getCredentialAccount;
 
-      console.debug(`  Enabling GET ${this.pathWithIdentifier}`);
+      console.debug(`  Enabling getCredentialAccount at GET ${this.pathWithIdentifier}`);
 
       router.get(this.pathWithIdentifier, async (req, res) => {
         if (!res.locals.credentials) {
@@ -355,7 +355,7 @@ export class Handler {
     if (this.handlers.acknowledgeWebhooks) {
       const handler = this.handlers.acknowledgeWebhooks;
 
-      console.debug(`  Enabling POST ${this.pathWithIdentifier}`);
+      console.debug(`  Enabling acknowledgeWebhooks at POST ${this.pathWithIdentifier}`);
 
       router.post(this.pathWithIdentifier, async (req, res) => {
         assertWebhookParseRequestPayload(req.body);
@@ -374,7 +374,7 @@ export class Handler {
     if (this.handlers.parseWebhooks) {
       const handler = this.handlers.parseWebhooks;
 
-      console.debug(`  Enabling POST ${this.pathWithIdentifier}`);
+      console.debug(`  Enabling parseWebhooks at POST ${this.pathWithIdentifier}`);
 
       router.post(this.pathWithIdentifier, async (req, res) => {
         assertWebhookParseRequestPayload(req.body);
@@ -393,7 +393,7 @@ export class Handler {
     if (this.handlers.updateWebhookSubscriptions) {
       const handler = this.handlers.updateWebhookSubscriptions;
 
-      console.debug(`  Enabling PUT ${this.pathWithIdentifier}`);
+      console.debug(`  Enabling updateWebhookSubscriptions at PUT ${this.pathWithIdentifier}`);
 
       router.put(this.pathWithIdentifier, async (req, res) => {
         if (!res.locals.credentials) {

package/src/integration.ts

@@ -18,6 +18,10 @@ function printErrorMessage(message: string) {
   console.error(message);
 }
 
+type Options = {
+  port?: number;
+};
+
 export default class Integration {
   private handlers: Handler[];
 
@@ -25,7 +29,10 @@ export default class Integration {
 
   private cache: Cache | undefined = undefined;
 
-  constructor() {
+  private port: number;
+
+  constructor(options: Options = {}) {
+    this.port = options.port || 9200;
     this.handlers = [];
   }
 
@@ -101,9 +108,8 @@ export default class Integration {
     // Must be the last handler.
     app.use(notFoundMiddleware);
 
-    this.instance = app.listen(process.env.PORT || 9200, () =>
-      console.info(`Server started on port ${process.env.PORT || 9200}.`),
-    );
+    // Start the server.
+    this.instance = app.listen(this.port, () => console.info(`Server started on port ${this.port}.`));
 
     // Trap exit signals.
     ['SIGTERM', 'SIGINT', 'SIGUSR2'].forEach(signalType => {

package/src/resources/provider.ts

@@ -1,5 +1,6 @@
 import { buildHttpError } from '../errors.js';
 import { Credentials } from '../middlewares/credentials.js';
+import Logger from '../resources/logger.js';
 
 /**
  * RateLimiter is a wrapper function that you can provide to limit the rate of calls to the provider based on the
@@ -18,25 +19,26 @@ import { Credentials } from '../middlewares/credentials.js';
  * @throws HttpError when the provider returns an error.
  */
 export type RateLimiter = <T>(
-  context: { credentials: Credentials },
+  context: { credentials: Credentials; logger: Logger },
   targetFunction: () => Promise<Response<T>>,
 ) => Promise<Response<T>>;
 
 export interface RequestOptions {
   credentials: Credentials;
+  logger: Logger;
   queryParams?: { [key: string]: string };
   additionnalheaders?: { [key: string]: string };
 }
 
 export interface Response<T> {
-  data: T;
+  body: T;
   status: number;
   headers: Headers;
 }
 
 export class Provider {
   protected rateLimiter: RateLimiter | undefined = undefined;
-  protected prepareRequest: (context: { credentials: Credentials }) => {
+  protected prepareRequest: (context: { credentials: Credentials; logger: Logger }) => {
     url: string;
     headers: Record<string, string>;
   };
@@ -116,7 +118,7 @@ export class Provider {
     body: Record<string, unknown> | null,
     options: RequestOptions & { defaultHeaders: { 'Content-Type': string; Accept: string }; method: string },
   ): Promise<Response<T>> {
-    const { url: providerUrl, headers: providerHeaders } = this.prepareRequest({ credentials: options.credentials });
+    const { url: providerUrl, headers: providerHeaders } = this.prepareRequest(options);
 
     let absoluteUrl = [providerUrl, endpoint.charAt(0) === '/' ? endpoint.substring(1) : endpoint].join('/');
 
@@ -149,8 +151,8 @@ export class Provider {
       }
 
       try {
-        const data: T = response.body ? await response.json() : undefined;
-        return { status: response.status, headers: response.headers, data };
+        const body: T = response.body ? await response.json() : undefined;
+        return { status: response.status, headers: response.headers, body };
       } catch {
         throw buildHttpError(400, 'Invalid JSON response');
       }

package/test/integration.test.ts

@@ -0,0 +1,34 @@
+import assert from 'node:assert/strict';
+import { describe, it } from 'node:test';
+import Integration from '../src/integration.js';
+
+describe('Integration', () => {
+  describe('constructor', () => {
+    it('defaults to port 9200', () => {
+      const integration = new Integration();
+
+      assert.equal(integration['port'], 9200);
+    });
+
+    it('accepts an optional port', () => {
+      const integration = new Integration({ port: 1234 });
+
+      assert.equal(integration['port'], 1234);
+    });
+  });
+
+  describe('addHandler', () => {
+    it('works', () => {
+      const integration = new Integration();
+
+      integration.addHandler('/', {});
+
+      const handler = integration['handlers'][0];
+
+      assert.ok(handler);
+      assert.equal(handler['path'], '/');
+      assert.equal(handler['pathWithIdentifier'], '/');
+      assert.deepEqual(handler['handlers'], {});
+    });
+  });
+});

package/test/middlewares/filters.test.ts

@@ -26,6 +26,23 @@ describe('filters middleware', () => {
     });
   });
 
+  it('decodes URI components', () => {
+    const request = { query: { filter: 'status=foo%2Cbar!!%2C%3Fbaz%3D!%3Equx' } } as express.Request<
+      // eslint-disable-next-line @typescript-eslint/no-explicit-any
+      any,
+      object,
+      object,
+      { filter: string }
+    >;
+    const response = { locals: {} } as express.Response;
+
+    middleware(request, response, () => {});
+
+    assert.deepEqual(response.locals, {
+      filters: [{ field: 'status', operator: OperatorType.EQUAL, values: ['foo,bar!!,?baz=!>qux'] }],
+    });
+  });
+
   it('no data', () => {
     const request = { query: {} } as express.Request;
     const response = { locals: {} } as express.Response;

package/test/resources/provider.test.ts

@@ -2,6 +2,7 @@ import assert from 'node:assert/strict';
 import { describe, it } from 'node:test';
 
 import { Provider } from '../../src/resources/provider.js';
+import Logger from '../../src/resources/logger.js';
 
 describe('Provider', () => {
   const provider = new Provider({
@@ -16,6 +17,8 @@ describe('Provider', () => {
     },
   });
 
+  const logger = new Logger();
+
   it('get', async context => {
     const response = new Response('{"data": "value"}', {
       status: 200,
@@ -26,6 +29,7 @@ describe('Provider', () => {
 
     const actualResponse = await provider.get('/endpoint', {
       credentials: { apiKey: 'apikey#1111' },
+      logger: logger,
       additionnalheaders: { 'X-Additional-Header': 'value1' },
     });
 
@@ -44,7 +48,7 @@ describe('Provider', () => {
         },
       },
     ]);
-    assert.deepEqual(actualResponse, { status: 200, headers: response.headers, data: { data: 'value' } });
+    assert.deepEqual(actualResponse, { status: 200, headers: response.headers, body: { data: 'value' } });
   });
 
   it('post with url encoded body', async context => {
@@ -62,6 +66,7 @@ describe('Provider', () => {
       },
       {
         credentials: { apiKey: 'apikey#1111' },
+        logger: logger,
         additionnalheaders: { 'X-Additional-Header': 'value1' },
       },
     );
@@ -81,7 +86,7 @@ describe('Provider', () => {
         },
       },
     ]);
-    assert.deepEqual(actualResponse, { status: 201, headers: response.headers, data: { data: 'value' } });
+    assert.deepEqual(actualResponse, { status: 201, headers: response.headers, body: { data: 'value' } });
   });
 
   it('put with json body', async context => {
@@ -100,6 +105,7 @@ describe('Provider', () => {
       },
       {
         credentials: { apiKey: 'apikey#1111' },
+        logger: logger,
         additionnalheaders: { 'X-Additional-Header': 'value1', 'Content-Type': 'application/json' },
       },
     );
@@ -119,7 +125,7 @@ describe('Provider', () => {
         },
       },
     ]);
-    assert.deepEqual(actualResponse, { status: 201, headers: response.headers, data: { data: 'value' } });
+    assert.deepEqual(actualResponse, { status: 201, headers: response.headers, body: { data: 'value' } });
   });
 
   it('patch with query params', async context => {
@@ -137,6 +143,7 @@ describe('Provider', () => {
       },
       {
         credentials: { apiKey: 'apikey#1111' },
+        logger: logger,
         queryParams: { param1: 'value1', param2: 'value2' },
         additionnalheaders: { 'X-Additional-Header': 'value1', 'Content-Type': 'application/json' },
       },
@@ -157,7 +164,7 @@ describe('Provider', () => {
         },
       },
     ]);
-    assert.deepEqual(actualResponse, { status: 201, headers: response.headers, data: { data: 'value' } });
+    assert.deepEqual(actualResponse, { status: 201, headers: response.headers, body: { data: 'value' } });
   });
 
   it('delete', async context => {
@@ -170,6 +177,7 @@ describe('Provider', () => {
 
     const actualResponse = await provider.delete('/endpoint/123', {
       credentials: { apiKey: 'apikey#1111' },
+      logger: logger,
       additionnalheaders: { 'X-Additional-Header': 'value1', 'Content-Type': 'application/json' },
     });
 
@@ -188,7 +196,7 @@ describe('Provider', () => {
         },
       },
     ]);
-    assert.deepEqual(actualResponse, { status: 204, headers: response.headers, data: undefined });
+    assert.deepEqual(actualResponse, { status: 204, headers: response.headers, body: undefined });
   });
 
   it('uses rate limiter if provided', async context => {
@@ -216,6 +224,7 @@ describe('Provider', () => {
 
     const options = {
       credentials: { apiKey: 'apikey#1111' },
+      logger: logger,
       additionnalheaders: { 'X-Additional-Header': 'value1', 'Content-Type': 'application/json' },
     };
 
@@ -238,7 +247,7 @@ describe('Provider', () => {
         },
       },
     ]);
-    assert.deepEqual(actualResponse, { status: 204, headers: response.headers, data: undefined });
+    assert.deepEqual(actualResponse, { status: 204, headers: response.headers, body: undefined });
   });
 
   it('throws on invalid json response', async context => {
@@ -251,6 +260,7 @@ describe('Provider', () => {
     assert.rejects(() =>
       provider.get('/endpoint/123', {
         credentials: { apiKey: 'apikey#1111' },
+        logger: logger,
       }),
     );
   });
@@ -265,6 +275,7 @@ describe('Provider', () => {
     assert.rejects(() =>
       provider.get('/endpoint/123', {
         credentials: { apiKey: 'apikey#1111' },
+        logger: logger,
       }),
     );
   });
@@ -279,6 +290,7 @@ describe('Provider', () => {
     assert.rejects(() =>
       provider.get('/endpoint/123', {
         credentials: { apiKey: 'apikey#1111' },
+        logger: logger,
       }),
     );
   });