@cubejs-client/core 1.8.3 → 1.8.5

package/dist/src/HttpTransport.js

@@ -27,7 +27,6 @@ export class HttpTransport {
         const requestMethod = method ?? this.method ?? (url.length < 2000 ? 'GET' : 'POST');
         if (requestMethod === 'POST') {
             url = `${this.apiUrl}/${apiMethod}`;
-            this.headers['Content-Type'] = 'application/json';
         }
         const effectiveFetchTimeout = fetchTimeout ?? this.fetchTimeout;
         const actualSignal = signal ||
@@ -43,6 +42,7 @@ export class HttpTransport {
                 Authorization: this.authorization,
                 'x-request-id': baseRequestId && `${baseRequestId}-span-${spanCounter++}`,
                 ...this.headers,
+                ...(requestMethod === 'POST' ? { 'Content-Type': 'application/json' } : {}),
             },
             credentials: this.credentials,
             body: requestMethod === 'POST' ? JSON.stringify(params) : null,
@@ -88,7 +88,6 @@ export class HttpTransport {
         const requestMethod = method ?? this.method ?? 'POST';
         if (requestMethod === 'POST') {
             url = `${this.apiUrl}/${apiMethod}`;
-            this.headers['Content-Type'] = 'application/json';
         }
         const effectiveFetchTimeout = fetchTimeout ?? this.fetchTimeout;
         let controller;
@@ -106,6 +105,7 @@ export class HttpTransport {
                         Authorization: this.authorization,
                         'x-request-id': baseRequestId || 'stream-request',
                         ...this.headers,
+                        ...(requestMethod === 'POST' ? { 'Content-Type': 'application/json' } : {}),
                     },
                     credentials: this.credentials,
                     body: requestMethod === 'POST' ? JSON.stringify(params || {}) : null,

package/dist/src/streaming.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"streaming.d.ts","sourceRoot":"","sources":["../../src/streaming.ts"],"names":[],"mappings":"AAAA,wBAAuB,cAAc,CAAC,GAAG,EAAE,QAAQ,GAAG,aAAa,CAAC,UAAU,CAAC,CAgB9E"}
+{"version":3,"file":"streaming.d.ts","sourceRoot":"","sources":["../../src/streaming.ts"],"names":[],"mappings":"AAAA,wBAAuB,cAAc,CAAC,GAAG,EAAE,QAAQ,GAAG,aAAa,CAAC,UAAU,CAAC,CAoB9E"}

package/dist/src/streaming.js

@@ -6,10 +6,12 @@ export async function* responseChunks(res) {
     try {
         while (true) {
             const { done, value } = await reader.read();
-            if (done)
+            if (done) {
                 break;
-            if (value)
+            }
+            if (value) {
                 yield value;
+            }
         }
     }
     finally {

package/dist/test/HttpTransport.test.js

@@ -10,8 +10,9 @@ describe('HttpTransport', () => {
     const queryUrlEncoded = '%7B%22measures%22%3A%5B%22Orders.count%22%5D%2C%22dimensions%22%3A%5B%22Users.country%22%5D%7D';
     const queryJson = '{"query":{"measures":["Orders.count"],"dimensions":["Users.country"]}}';
     const ids = [];
-    for (let i = 0; i < 40; i++)
+    for (let i = 0; i < 40; i++) {
         ids.push('a40b2052-4137-11eb-b378-0242ac130002');
+    }
     const LargeQuery = {
         measures: ['Orders.count'],
         dimensions: ['Users.country'],
@@ -129,6 +130,19 @@ describe('HttpTransport', () => {
             body: largeQueryJson
         });
     });
+    test('it does not add Content-Type to the shared headers on POST', async () => {
+        const headers = { 'X-Custom': 'value' };
+        const transport = new HttpTransport({
+            authorization: 'token',
+            apiUrl,
+            headers,
+        });
+        await transport.request('load', { query: LargeQuery }).subscribe(() => undefined);
+        await transport.request('load', { query }).subscribe(() => undefined);
+        expect(headers).toEqual({ 'X-Custom': 'value' });
+        expect(mockedFetch.mock.calls[0]?.[1]?.headers).toMatchObject({ 'Content-Type': 'application/json' });
+        expect(mockedFetch.mock.calls[1]?.[1]?.headers).not.toHaveProperty('Content-Type');
+    });
     // Signal tests from src/tests/HttpTransport.test.js
     describe('Signal functionality', () => {
         beforeEach(() => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cubejs-client/core",
-  "version": "1.8.3",
+  "version": "1.8.5",
   "engines": {
     "node": ">=20.0.0"
   },
@@ -57,7 +57,7 @@
   ],
   "license": "MIT",
   "devDependencies": {
-    "@cubejs-backend/linter": "1.8.3",
+    "@cubejs-backend/linter": "1.8.5",
     "@types/d3-format": "^3",
     "@types/d3-time-format": "^4",
     "@types/moment-range": "^4.0.0",
@@ -66,5 +66,5 @@
     "typescript": "~6.0.3",
     "vitest": "^4"
   },
-  "gitHead": "e15efc589083e85b0912056bea6d1ae2eb68c720"
+  "gitHead": "d01994a71e64f2e8074ee04fefa3a0d9f369cc81"
 }