@journeyapps/cloudcode - npm Package Compare versions

Comparing version 0.0.0-dev.c4ff64b to 0.0.0-dev.c5ab339

dist/bootstrap.js

@@ -43,3 +43,3 @@ "use strict";
         processId: exports.PROCESS_ID,
-        message: `Start process. Process trace ID: ${exports.PROCESS_ID}`
+        message: `Start process. Process trace ID: ${exports.PROCESS_ID}. Revision: ${global.CloudCode.deployInfo.description}.`
     });
@@ -46,0 +46,0 @@ // Default Lambda uncaughtException handler does a process.exit(1), preventing us from logging the error.

dist/busycheck.js

@@ -20,3 +20,2 @@ "use strict";
     }
-    let blacklisted = false;
     for (let re of HOST_BLACKLIST) {
@@ -37,7 +36,7 @@ if (re.test(host)) {
  *
- * @return {string[]} array of descriptions of open sockets
+ * @return array of descriptions of open sockets
  */
 function getActiveHandles() {
-    var results = [];
-    var handles = process._getActiveHandles();
+    let results = [];
+    const handles = process._getActiveHandles();
     for (let handle of handles) {
@@ -62,5 +61,14 @@ if (isSocket(handle)) {
             if (socket._httpMessage) {
+                if (socket._httpMessage.res && !socket._httpMessage.res.readable) {
+                    // Alternative: `socket._httpMessage.res?._readableState?.ended`
+                    // Even though the socket is still open, the response has been read.
+                    // This is typically the case if the developer did `await response.text()`. This is good enough.
+                    // Alternatively, we could say that even just `await fetch(...)` is good enough. The presence of `.res`
+                    // would be a good indication of that.
+                    continue;
+                }
                 message = `HTTP ${socket._httpMessage.method} ${socket._host} ${socket._httpMessage.path}`;
             }
             else if (socket._host) {
+                // Note: This could include HTTP requests, where the request hasn't been sent yet.
                 message = `Socket ${socket._host}`;
@@ -81,3 +89,3 @@ }
 function busycheck() {
-    var handles = getActiveHandles();
+    const handles = getActiveHandles();
     if (handles.length > 0) {
@@ -84,0 +92,0 @@ let msg = 'Some requests are still active, and block the response until they complete:';

dist/CloudCodeEnvironment.d.ts

@@ -18,4 +18,11 @@ import { Database } from '@journeyapps/db';
 }
+export interface DeployInfo {
+    readonly revision?: string;
+    readonly draftId?: string;
+    readonly branch: string;
+    readonly description: string;
+}
 export declare class CloudCodeEnvironment implements EventSources {
     readonly task: TaskEnvironment;
+    readonly deployInfo: DeployInfo;
     APP: string;
@@ -22,0 +29,0 @@ EDITOR_TEST: string;

dist/CloudCodeEnvironment.js

@@ -20,2 +20,19 @@ "use strict";
         this.task = task;
+        const revision = process.env.CLOUDCODE_REVISION || 'unknown';
+        const branch = process.env.CLOUDCODE_BRANCH || 'unknown';
+        const draftId = process.env.CLOUDCODE_DRAFT_ID;
+        if (draftId) {
+            this.deployInfo = {
+                draftId: draftId,
+                branch: branch,
+                description: `${branch}/draft/${draftId}`
+            };
+        }
+        else {
+            this.deployInfo = {
+                revision: revision,
+                branch: branch,
+                description: `${branch}/${revision}`
+            };
+        }
     }
@@ -22,0 +39,0 @@ setTaskContext(context) {

dist/entry.js

@@ -89,3 +89,3 @@ 'use strict';
         type: rawRequest.eventSource,
-        message: `Start '${rawRequest.eventSource}' request. Process trace ID: '${bootstrap_1.PROCESS_ID}'`
+        message: `Start '${rawRequest.eventSource}' request. Process trace ID: '${bootstrap_1.PROCESS_ID}'. Revision: '${global.CloudCode.deployInfo.description}'.`
     });
@@ -200,3 +200,2 @@ // TODO: Remove these non-webby bits
 }
-
 Found module:
@@ -203,0 +202,0 @@ ${util.format(task.module)}

package.json

 {
   "name": "@journeyapps/cloudcode",
-  "version": "0.0.0-dev.c4ff64b",
+  "version": "0.0.0-dev.c5ab339",
   "main": "./dist/index",
   "types": "./dist/index",
   "license": "MIT",
+  "exports": {
+    ".": "./dist/index.js",
+    "./entry": "./dist/entry.js",
+    "./*": "./*"
+  },
   "files": [
@@ -8,0 +13,0 @@ "README.md",

New alerts

Environment variable access

Supply chain risk

Package accesses environment variables, which may be a sign of credential stuffing or data theft.

Found 3 instances in 1 package

Improved metrics

Total package byte prevSize

158710

increased by1.51%

Lines of code

2772

increased by1.17%

Worsened metrics

Number of low supply chain risk alerts

12

increased by33.33%

No dependency changes