cloudflare-tcp-fetcher - npm Package Compare versions

Comparing version 0.9.0 to 0.9.1

example/package.json

@@ -14,3 +14,6 @@ {
     "wrangler": "^3.0.0"
+  },
+  "dependencies": {
+    "cloudflare-tcp-fetcher": "^0.9.0"
   }
 }

example/src/index.ts

@@ -1,9 +0,7 @@
-import { fetchUsingTcp } from "../../src/index";
+import { fetchUsingTcp } from "cloudflare-tcp-fetcher";
 
 export default {
 	async fetch(request: Request, env: never, ctx: ExecutionContext): Promise<Response> {
-		const response = await fetchUsingTcp("https://google.com");
-
-		return response;
+		return fetchUsingTcp("https://google.com");
 	},
 };

package.json

 {
   "name": "cloudflare-tcp-fetcher",
-  "version": "0.9.0",
+  "version": "0.9.1",
   "description": "",
@@ -5,0 +5,0 @@ "main": "src/index.js",

src/client/TcpFetcher.ts

@@ -21,2 +21,4 @@ import { connect } from 'cloudflare:sockets';
     return new Promise((resolve, reject) => {
+      let resolved = false;
+
       socket.opened.then(async () => {
@@ -60,2 +62,4 @@ const writer = socket.writable.getWriter();
                 if(response.bodyData.length >= parseInt(response.headers["content-length"])) {
+                  resolved = true;
+
                   resolve(new Response(response.bodyData, {
@@ -78,2 +82,4 @@ status: response.statusCode,
           else {
+            resolved = true;
+
             reject("Reader was finished.");
@@ -84,8 +90,14 @@ }
         reader.read().then(handleRead);
-      }, reject);
+      }, (reason) => {
+        resolved = true;
+        
+        reject(reason);
+      });
 
       setTimeout(() => {
-        socket.close();
+        if(!resolved) {
+          socket.close();
 
-        reject("Connection timed out (>30s)");
+          reject("Connection timed out (>30s)");
+        }
       }, 30000);
@@ -92,0 +104,0 @@ });

Fixed alerts

No README

Quality

Package does not have a README. This may indicate a failed publish or a low quality package.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

102906

increased by17.48%

Number of medium quality alerts

1

decreased by-50%

Number of lines in readme file

43

increased byInfinity%

Worsened metrics

Number of package files

16

decreased by-46.67%

Lines of code

1629

decreased by-8.84%

No dependency changes