got 14.6.1 → 14.6.2

package/dist/source/core/index.d.ts

@@ -107,7 +107,7 @@ export default class Request extends Duplex implements RequestEvents<Request> {
     private _flushed;
     private _aborted;
     private _expectedContentLength?;
-    private _byteCounter?;
+    private _compressedBytesCount?;
     private readonly _requestId;
     private _requestInitialized;
     constructor(url: UrlType, options?: OptionsType, defaults?: DefaultsType);

package/dist/source/core/index.js

@@ -1,7 +1,8 @@
 import process from 'node:process';
 import { Buffer } from 'node:buffer';
-import { Duplex, Transform } from 'node:stream';
+import { Duplex } from 'node:stream';
 import http, { ServerResponse } from 'node:http';
+import { byteLength } from 'byte-counter';
 import timer from '@szmarczak/http-timer';
 import CacheableRequest, { CacheError as CacheableCacheError, } from 'cacheable-request';
 import decompressResponse from 'decompress-response';
@@ -37,17 +38,6 @@ const proxiedRequestEvents = [
     'upgrade',
 ];
 const noop = () => { };
-/**
-Stream transform that counts bytes passing through.
-Used to track compressed bytes before decompression for content-length validation.
-*/
-class ByteCounter extends Transform {
-    count = 0;
-    _transform(chunk, _encoding, callback) {
-        this.count += chunk.length;
-        callback(null, chunk);
-    }
-}
 export default class Request extends Duplex {
     // @ts-expect-error - Ignoring for now.
     ['constructor'];
@@ -76,7 +66,7 @@ export default class Request extends Duplex {
     _flushed = false;
     _aborted = false;
     _expectedContentLength;
-    _byteCounter;
+    _compressedBytesCount;
     _requestId = generateRequestId();
     // We need this because `this._request` if `undefined` when using cache
     _requestInitialized = false;
@@ -473,9 +463,9 @@ export default class Request extends Duplex {
     }
     _checkContentLengthMismatch() {
         if (this.options.strictContentLength && this._expectedContentLength !== undefined) {
-            // Use ByteCounter's count when available (for compressed responses),
+            // Use compressed bytes count when available (for compressed responses),
             // otherwise use _downloadedSize (for uncompressed responses)
-            const actualSize = this._byteCounter?.count ?? this._downloadedSize;
+            const actualSize = this._compressedBytesCount ?? this._downloadedSize;
             if (actualSize !== this._expectedContentLength) {
                 this._beforeError(new ReadError({
                     message: `Content-Length mismatch: expected ${this._expectedContentLength} bytes, received ${actualSize} bytes`,
@@ -578,9 +568,9 @@ export default class Request extends Duplex {
             // When strictContentLength is enabled, track compressed bytes by listening to
             // the native response's data events before decompression
             if (options.strictContentLength) {
-                this._byteCounter = new ByteCounter();
+                this._compressedBytesCount = 0;
                 this._nativeResponse.on('data', (chunk) => {
-                    this._byteCounter.count += chunk.length;
+                    this._compressedBytesCount += byteLength(chunk);
                 });
             }
             response = decompressResponse(response);
@@ -1259,7 +1249,9 @@ export default class Request extends Duplex {
         this._request.write(chunk, encoding, (error) => {
             // The `!destroyed` check is required to prevent `uploadProgress` being emitted after the stream was destroyed
             if (!error && !this._request.destroyed) {
-                this._uploadedSize += Buffer.byteLength(chunk, encoding);
+                // For strings, encode them first to measure the actual bytes that will be sent
+                const bytes = typeof chunk === 'string' ? Buffer.from(chunk, encoding) : chunk;
+                this._uploadedSize += byteLength(bytes);
                 const progress = this.uploadProgress;
                 if (progress.percent < 1) {
                     this.emit('uploadProgress', progress);

package/dist/source/core/options.d.ts

@@ -1528,7 +1528,6 @@ export default class Options {
         _defaultAgent?: http.Agent | undefined;
         auth?: string | null | undefined;
         defaultPort?: number | string | undefined;
-        hints?: import("dns").LookupOptions["hints"];
         host?: string | null | undefined;
         hostname?: string | null | undefined;
         insecureHTTPParser?: boolean | undefined;
@@ -1540,7 +1539,8 @@ export default class Options {
         signal?: AbortSignal | undefined;
         socketPath?: string | undefined;
         uniqueHeaders?: Array<string | string[]> | undefined;
-        joinDuplicateHeaders?: boolean;
+        joinDuplicateHeaders?: boolean | undefined;
+        hints?: number | undefined;
         ALPNCallback?: ((arg: {
             servername: string;
             protocols: string[];

package/dist/source/core/options.js

@@ -781,7 +781,7 @@ export default class Options {
         }
         // Detect if URL is already absolute (has a protocol/scheme)
         const valueString = value.toString();
-        const isAbsolute = is.urlInstance(value) || /^[a-z][a-z\d+.-]*:/i.test(valueString);
+        const isAbsolute = is.urlInstance(value) || /^[a-z][a-z\d+.-]*:\/\//i.test(valueString);
         // Only concatenate prefixUrl if the URL is relative
         const urlString = isAbsolute ? valueString : `${this.prefixUrl}${valueString}`;
         const url = new URL(urlString);

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "got",
-	"version": "14.6.1",
+	"version": "14.6.2",
 	"description": "Human-friendly and powerful HTTP request library for Node.js",
 	"license": "MIT",
 	"repository": "sindresorhus/got",
@@ -53,6 +53,7 @@
 	"dependencies": {
 		"@sindresorhus/is": "^7.0.1",
 		"@szmarczak/http-timer": "^5.0.1",
+		"byte-counter": "^0.1.0",
 		"cacheable-lookup": "^7.0.0",
 		"cacheable-request": "^13.0.12",
 		"decompress-response": "^10.0.0",