@trayio/cdk-runtime 5.26.0 → 5.28.0

package/dist/connector/polling/PollingServiceClient.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"PollingServiceClient.d.ts","sourceRoot":"","sources":["../../../src/connector/polling/PollingServiceClient.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,UAAU,EAAE,MAAM,iCAAiC,CAAC;AAE7D,OAAO,EAGN,eAAe,EAEf,MAAM,kDAAkD,CAAC;AAuB1D,MAAM,MAAM,6BAA6B,GAAG;IAC3C,aAAa,EAAE,MAAM,CAAC;IACtB,gBAAgB,EAAE,MAAM,CAAC;IACzB,qBAAqB,EAAE,MAAM,CAAC;IAC9B,gBAAgB,EAAE,MAAM,GAAG,IAAI,CAAC;IAChC,cAAc,EAAE,MAAM,GAAG,IAAI,CAAC;IAC9B,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,SAAS,EAAE,MAAM,GAAG,IAAI,CAAC;IACzB,SAAS,EAAE,MAAM,CAAC;IAClB,eAAe,CAAC,EAAE,eAAe,CAAC;IAClC,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,aAAa,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC9B;;;;;OAKG;IACH,cAAc,EAAE,OAAO,CAAC;CACxB,CAAC;AAEF,MAAM,MAAM,+BAA+B,GAAG;IAC7C,aAAa,EAAE,MAAM,CAAC;IACtB,gBAAgB,EAAE,MAAM,CAAC;IACzB;;;;OAIG;IACH,qBAAqB,EAAE,MAAM,CAAC;IAC9B,SAAS,EAAE,MAAM,CAAC;IAClB;;;;;OAKG;IACH,cAAc,EAAE,OAAO,CAAC;CACxB,CAAC;AA4BF;;;;GAIG;AACH,qBAAa,oBAAoB;IACpB,OAAO,CAAC,UAAU;gBAAV,UAAU,EAAE,UAAU;IAEpC,QAAQ,CACb,OAAO,EAAE,6BAA6B,EACtC,cAAc,EAAE,MAAM,GACpB,OAAO,CAAC,IAAI,CAAC;IAmCV,UAAU,CACf,OAAO,EAAE,+BAA+B,EACxC,cAAc,EAAE,MAAM,GACpB,OAAO,CAAC,IAAI,CAAC;YAaF,QAAQ;CAgCtB"}
+{"version":3,"file":"PollingServiceClient.d.ts","sourceRoot":"","sources":["../../../src/connector/polling/PollingServiceClient.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,UAAU,EAAE,MAAM,iCAAiC,CAAC;AAE7D,OAAO,EAGN,eAAe,EAEf,MAAM,kDAAkD,CAAC;AAuB1D,MAAM,MAAM,6BAA6B,GAAG;IAC3C,aAAa,EAAE,MAAM,CAAC;IACtB,gBAAgB,EAAE,MAAM,CAAC;IACzB,qBAAqB,EAAE,MAAM,CAAC;IAC9B,gBAAgB,EAAE,MAAM,GAAG,IAAI,CAAC;IAChC,cAAc,EAAE,MAAM,GAAG,IAAI,CAAC;IAC9B,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,SAAS,EAAE,MAAM,GAAG,IAAI,CAAC;IACzB,SAAS,EAAE,MAAM,CAAC;IAClB,eAAe,CAAC,EAAE,eAAe,CAAC;IAClC,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,aAAa,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC9B;;;;;OAKG;IACH,cAAc,EAAE,OAAO,CAAC;CACxB,CAAC;AAEF,MAAM,MAAM,+BAA+B,GAAG;IAC7C,aAAa,EAAE,MAAM,CAAC;IACtB,gBAAgB,EAAE,MAAM,CAAC;IACzB;;;;OAIG;IACH,qBAAqB,EAAE,MAAM,CAAC;IAC9B,SAAS,EAAE,MAAM,CAAC;IAClB;;;;;OAKG;IACH,cAAc,EAAE,OAAO,CAAC;CACxB,CAAC;AAyBF;;;;GAIG;AACH,qBAAa,oBAAoB;IACpB,OAAO,CAAC,UAAU;gBAAV,UAAU,EAAE,UAAU;IAEpC,QAAQ,CACb,OAAO,EAAE,6BAA6B,EACtC,cAAc,EAAE,MAAM,GACpB,OAAO,CAAC,IAAI,CAAC;IAmCV,UAAU,CACf,OAAO,EAAE,+BAA+B,EACxC,cAAc,EAAE,MAAM,GACpB,OAAO,CAAC,IAAI,CAAC;YAaF,QAAQ;CAqCtB"}

package/dist/connector/polling/PollingServiceClient.integration.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=PollingServiceClient.integration.test.d.ts.map

package/dist/connector/polling/PollingServiceClient.integration.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"PollingServiceClient.integration.test.d.ts","sourceRoot":"","sources":["../../../src/connector/polling/PollingServiceClient.integration.test.ts"],"names":[],"mappings":""}

package/dist/connector/polling/PollingServiceClient.integration.test.js

@@ -0,0 +1,130 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || function (mod) {
+    if (mod && mod.__esModule) return mod;
+    var result = {};
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    __setModuleDefault(result, mod);
+    return result;
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const http = __importStar(require("http"));
+const AxiosHttpClient_1 = require("@trayio/axios/http/AxiosHttpClient");
+const PollingServiceClient_1 = require("./PollingServiceClient");
+const ENV_VAR = 'TRAY_POLLING_SERVICE_URL';
+/**
+ * Emulates the production path that broke deregistration: a hop that only
+ * honours a request body on DELETE when Content-Length declares it, so a
+ * chunked DELETE body arrives at the polling service as empty. The 400
+ * response mirrors the polling service's Tapir decode failure verbatim.
+ */
+const startPollingServiceStub = async (observed) => {
+    const server = http.createServer((req, res) => {
+        const chunks = [];
+        req.on('data', (chunk) => chunks.push(chunk));
+        req.on('end', () => {
+            const received = Buffer.concat(chunks).toString('utf8');
+            const bodyHonoured = req.method !== 'DELETE' || req.headers['content-length'] !== undefined;
+            const body = bodyHonoured ? received : '';
+            observed.push({ method: req.method, headers: req.headers, body });
+            try {
+                JSON.parse(body);
+                res.writeHead(200, { 'Content-Type': 'application/json' });
+                res.end('{}');
+            }
+            catch {
+                res.writeHead(400, { 'Content-Type': 'application/json' });
+                res.end(JSON.stringify({
+                    message: 'Invalid value for: body (exhausted input)',
+                }));
+            }
+        });
+    });
+    await new Promise((resolve) => {
+        server.listen(0, '127.0.0.1', () => resolve());
+    });
+    return server;
+};
+describe('PollingServiceClient against a DELETE-body-hostile HTTP hop', () => {
+    let server;
+    let observed;
+    let originalEnv;
+    beforeAll(async () => {
+        observed = [];
+        server = await startPollingServiceStub(observed);
+        originalEnv = process.env[ENV_VAR];
+        const { port } = server.address();
+        process.env[ENV_VAR] = `http://127.0.0.1:${port}`;
+    });
+    afterAll(async () => {
+        if (originalEnv === undefined) {
+            delete process.env[ENV_VAR];
+        }
+        else {
+            process.env[ENV_VAR] = originalEnv;
+        }
+        await new Promise((resolve, reject) => {
+            server.close((err) => (err ? reject(err) : resolve()));
+        });
+    });
+    beforeEach(() => {
+        observed.length = 0;
+    });
+    it('deregister body survives: sent with Content-Length, not chunked, and decodes server-side', async () => {
+        const client = new PollingServiceClient_1.PollingServiceClient(new AxiosHttpClient_1.AxiosHttpClient());
+        await client.deregister({
+            connectorName: 'sample-polling-trigger',
+            connectorVersion: '1.0',
+            registeredHandlerName: 'new_rows_destroy',
+            publicUrl: 'https://routing-token.staging.trayapp.io',
+            preserveCursor: true,
+        }, 'exec-token');
+        expect(observed).toHaveLength(1);
+        const request = observed[0];
+        expect(request.method).toBe('DELETE');
+        expect(request.headers['content-length']).toBeDefined();
+        expect(request.headers['transfer-encoding']).toBeUndefined();
+        expect(JSON.parse(request.body)).toEqual({
+            connectorName: 'sample-polling-trigger',
+            connectorVersion: '1.0',
+            operationName: 'new_rows',
+            publicUrl: 'https://routing-token.staging.trayapp.io',
+            preserveCursor: true,
+        });
+    });
+    it('register keeps working through the same hop', async () => {
+        const client = new PollingServiceClient_1.PollingServiceClient(new AxiosHttpClient_1.AxiosHttpClient());
+        await client.register({
+            connectorName: 'sample-polling-trigger',
+            connectorVersion: '1.0',
+            registeredHandlerName: 'new_rows',
+            authenticationId: null,
+            organizationId: 'org-uuid',
+            workspaceId: 'ws-uuid',
+            projectId: null,
+            publicUrl: 'https://routing-token.staging.trayapp.io',
+            operationInput: { pageSize: 2 },
+        }, 'exec-token');
+        expect(observed).toHaveLength(1);
+        expect(observed[0].method).toBe('POST');
+        expect(JSON.parse(observed[0].body)).toMatchObject({
+            operationName: 'new_rows',
+            operationInput: { pageSize: 2 },
+        });
+    });
+});

package/dist/connector/polling/PollingServiceClient.js

@@ -24,8 +24,8 @@ var __importStar = (this && this.__importStar) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.PollingServiceClient = void 0;
-const stream_1 = require("stream");
 const E = __importStar(require("fp-ts/Either"));
+const BufferExtensions_1 = require("@trayio/commons/buffer/BufferExtensions");
 const Http_1 = require("@trayio/commons/http/Http");
 const PollingTypes_1 = require("@trayio/cdk-dsl/connector/operation/PollingTypes");
 const POLLING_SERVICE_URL_ENV = 'TRAY_POLLING_SERVICE_URL';
@@ -61,7 +61,6 @@ const resolveEndpoint = () => {
     }
     return `${baseUrl.replace(/\/$/, '')}${TRIGGERS_REGISTER_PATH}`;
 };
-const bodyToReadable = (payload) => stream_1.Readable.from(JSON.stringify(payload));
 /**
  * Internal HTTP client used by the runtime when invoking polling trigger
  * create / destroy operations. Not exported from the public `@trayio/cdk-runtime`
@@ -116,12 +115,17 @@ class PollingServiceClient {
             throw new Error('polling service call requires an execution token (ctx.executionToken)');
         }
         const url = resolveEndpoint();
+        // A buffered body with an explicit Content-Length: streamed bodies go out
+        // with Transfer-Encoding: chunked, and chunked DELETE bodies get dropped
+        // in transit, so the polling service receives an empty body and responds 400.
+        const bodyBuffer = Buffer.from(JSON.stringify(body), 'utf8');
         const request = Http_1.HttpRequest.create({
             headers: {
                 [Http_1.HttpHeader.ContentType]: 'application/json',
+                [Http_1.HttpHeader.ContentLength]: String(bodyBuffer.byteLength),
                 [EXECUTION_TOKEN_HEADER]: executionToken,
             },
-            body: bodyToReadable(body),
+            body: BufferExtensions_1.BufferExtensions.bufferToArrayBuffer(bodyBuffer),
         });
         const responseE = await this.httpClient.execute(method, url, request)();
         if (E.isLeft(responseE)) {

package/dist/connector/polling/PollingServiceClient.unit.test.js

@@ -33,16 +33,13 @@ const ENDPOINT_PATH = '/triggers/register';
 const ENV_VAR = 'TRAY_POLLING_SERVICE_URL';
 const createMockHttpClient = (capture) => ({
     execute: jest.fn().mockImplementation((method, url, request) => TE.tryCatch(async () => {
-        const chunks = [];
-        for await (const chunk of request.body) {
-            chunks.push(typeof chunk === 'string' ? Buffer.from(chunk) : chunk);
-        }
-        const raw = Buffer.concat(chunks).toString('utf8');
+        const raw = Buffer.from(request.body).toString('utf8');
         capture.push({
             method,
             url,
             headers: request.headers,
             body: raw.length > 0 ? JSON.parse(raw) : undefined,
+            bodyByteLength: Buffer.byteLength(raw, 'utf8'),
         });
         return {
             statusCode: 200,
@@ -107,6 +104,7 @@ describe('PollingServiceClient', () => {
             expect(call.url).toBe(`${BASE_URL}${ENDPOINT_PATH}`);
             expect(call.headers).toMatchObject({
                 'Content-Type': 'application/json',
+                'Content-Length': String(call.bodyByteLength),
                 'X-Tray-Execution-Token': 'exec-jwt-token',
             });
             expect(call.body).toEqual({
@@ -239,8 +237,11 @@ describe('PollingServiceClient', () => {
             expect(call.method).toBe(Http_1.HttpMethod.Delete);
             expect(call.url).toBe(`${BASE_URL}${ENDPOINT_PATH}`);
             expect(call.headers).toMatchObject({
+                'Content-Type': 'application/json',
+                'Content-Length': String(call.bodyByteLength),
                 'X-Tray-Execution-Token': 'exec-jwt-token',
             });
+            expect(call.bodyByteLength).toBeGreaterThan(0);
             expect(call.body).toEqual({
                 connectorName: 'salesforce',
                 connectorVersion: '7.2.0',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@trayio/cdk-runtime",
-  "version": "5.26.0",
+  "version": "5.28.0",
   "description": "A Runtime that executes connector operations defined using the CDK DSL",
   "exports": {
     "./*": "./dist/*.js"
@@ -14,10 +14,10 @@
     "node": ">=18.x"
   },
   "dependencies": {
-    "@trayio/axios": "5.26.0",
-    "@trayio/cdk-dsl": "5.26.0",
-    "@trayio/express": "5.26.0",
-    "@trayio/winston": "5.26.0",
+    "@trayio/axios": "5.28.0",
+    "@trayio/cdk-dsl": "5.28.0",
+    "@trayio/express": "5.28.0",
+    "@trayio/winston": "5.28.0",
     "mime": "3.0.0",
     "uuid": "9.0.0"
   },