@milaboratories/pl-drivers 1.8.2 → 1.9.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@milaboratories/pl-drivers",
-  "version": "1.8.2",
+  "version": "1.9.0",
   "engines": {
     "node": ">=20"
   },
@@ -22,21 +22,21 @@
   "dependencies": {
     "decompress": "^4.2.1",
     "@grpc/grpc-js": "~1.13.4",
-    "@protobuf-ts/grpc-transport": "2.11.0",
-    "@protobuf-ts/runtime": "2.11.0",
-    "@protobuf-ts/runtime-rpc": "2.11.0",
-    "@protobuf-ts/plugin": "2.11.0",
+    "@protobuf-ts/grpc-transport": "2.11.1",
+    "@protobuf-ts/runtime": "2.11.1",
+    "@protobuf-ts/runtime-rpc": "2.11.1",
+    "@protobuf-ts/plugin": "2.11.1",
     "denque": "^2.1.0",
     "tar-fs": "^3.0.9",
-    "undici": "~7.10.0",
+    "undici": "~7.13.0",
     "zod": "~3.23.8",
     "upath": "^2.0.1",
-    "@milaboratories/ts-helpers": "^1.4.3",
     "@milaboratories/helpers": "^1.6.19",
+    "@milaboratories/ts-helpers": "^1.4.3",
+    "@milaboratories/pl-tree": "^1.7.5",
+    "@milaboratories/pl-model-common": "^1.19.4",
     "@milaboratories/computable": "^2.6.3",
-    "@milaboratories/pl-client": "^2.11.6",
-    "@milaboratories/pl-model-common": "^1.19.3",
-    "@milaboratories/pl-tree": "^1.7.5"
+    "@milaboratories/pl-client": "^2.11.7"
   },
   "devDependencies": {
     "eslint": "^9.25.1",

package/src/clients/upload.ts

@@ -76,11 +76,32 @@ export class ClientUpload {
     const chunk = await readFileChunk(path, info.chunkStart, info.chunkEnd);
     await checkExpectedMTime(path, expectedMTimeUnix);
 
+    const contentLength = Number(info.chunkEnd - info.chunkStart);
+    if (chunk.length !== contentLength) {
+      throw new Error(
+        `Chunk size mismatch: expected ${contentLength} bytes, but read ${chunk.length} bytes from file`,
+      );
+    }
+
+    const headers = Object.fromEntries(info.headers.map(({ name, value }) => [name, value]));
+
+    const contentLengthKey = Object.keys(headers).find((key) => key.toLowerCase() === 'content-length');
+    if (contentLengthKey) {
+      const existingContentLength = Number(headers[contentLengthKey]);
+      if (existingContentLength !== contentLength) {
+        throw new Error(
+          `Content-Length mismatch: expected ${contentLength}, but got ${existingContentLength} in headers`,
+        );
+      }
+    }
+
+    // content length will be automatically added by undici, so we don't need to set it here
+
     try {
       const {
         body: rawBody,
         statusCode,
-        headers,
+        headers: responseHeaders,
       } = await request(info.uploadUrl, {
         dispatcher: this.httpClient,
         body: chunk,
@@ -90,13 +111,17 @@ export class ClientUpload {
         // that's why we got big timeout here.
         headersTimeout: 60000,
         bodyTimeout: 60000,
-        headers: Object.fromEntries(info.headers.map(({ name, value }) => [name, value])),
+        // Prevent connection reuse by setting "Connection: close" header.
+        // This works around an issue with the backend's built-in S3 implementation
+        // that caused HTTP/1.1 protocol lines to be included in the uploaded file content.
+        reset: true,
+        headers,
         method: info.method.toUpperCase() as Dispatcher.HttpMethod,
       });
 
       // always read the body for resources to be garbage collected.
       const body = await rawBody.text();
-      checkStatusCodeOk(statusCode, body, headers, info);
+      checkStatusCodeOk(statusCode, body, responseHeaders, info);
     } catch (e: unknown) {
       if (e instanceof NetworkError)
         throw e;