@based/get-service - npm Package Compare versions

Comparing version 0.1.7 to 0.2.0

dist/index.d.ts

@@ -0,1 +1,2 @@
+export declare const getClusterUrl: (cluster?: string) => string;
 export declare type ServiceUrl = {
@@ -2,0 +3,0 @@ protocol: string;

dist/index.js

@@ -6,3 +6,5 @@ "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.getClusterUrl = void 0;
 const cross_fetch_1 = __importDefault(require("cross-fetch"));
+const DEV_CLUSTER = 'https://dg194mksjrqfz.cloudfront.net';
 const parsUrlRe = /^(?:(tcp|wss?|https?):\/\/)?([a-z0-9.-]*)(?::(\d+))?$/;
@@ -12,10 +14,28 @@ const wait = (t = 1e3) => new Promise((resolve) => {
 });
-// this has to be fixed - has to come from machine env var (should be there)
+if (!process.env.SERVICE_SELECTOR_LIST && typeof window === 'undefined') {
+    if (process.env.CLUSTER === 'dev') {
+        process.env.SERVICE_SELECTOR_LIST = DEV_CLUSTER;
+    }
+}
 const SELECTOR_LIST = process.env.CLUSTER === 'local'
-    ? 'http://localhost:7022'
-    : // : process.env.SERVICE_SELECTOR_LIST
-        // ? `https://${process.env.SERVICE_SELECTOR_LIST}`
-        'https://dg194mksjrqfz.cloudfront.net';
+    ? `http://${process.env.SERVICE_SELECTOR_LIST}`
+    : `https://${process.env.SERVICE_SELECTOR_LIST}`;
 let registryIndex = 0;
+const getClusterUrl = (cluster) => {
+    if (!cluster) {
+        return DEV_CLUSTER;
+    }
+    if (cluster === 'local') {
+        return 'http://localhost:7022';
+    }
+    if (cluster === 'localSpesh') {
+        return 'http://localhost:9022';
+    }
+    return cluster;
+};
+exports.getClusterUrl = getClusterUrl;
 async function getService(opts, retries = 0, cluster = SELECTOR_LIST) {
+    if (!cluster) {
+        throw new Error('No cluster url defined');
+    }
     try {
@@ -22,0 +42,0 @@ const registryUrls = await Promise.race([

package.json

 {
   "name": "@based/get-service",
-  "version": "0.1.7",
+  "version": "0.2.0",
   "main": "dist/index.js",
@@ -5,0 +5,0 @@ "__module": "src/index.ts",

src/index.ts

 import fetch from 'cross-fetch'
 
+const DEV_CLUSTER = 'https://dg194mksjrqfz.cloudfront.net'
+
 const parsUrlRe = /^(?:(tcp|wss?|https?):\/\/)?([a-z0-9.-]*)(?::(\d+))?$/
@@ -9,12 +11,28 @@ const wait = (t = 1e3) =>
 
-// this has to be fixed - has to come from machine env var (should be there)
+if (!process.env.SERVICE_SELECTOR_LIST && typeof window === 'undefined') {
+  if (process.env.CLUSTER === 'dev') {
+    process.env.SERVICE_SELECTOR_LIST = DEV_CLUSTER
+  }
+}
+
 const SELECTOR_LIST =
   process.env.CLUSTER === 'local'
-    ? 'http://localhost:7022'
-    : // : process.env.SERVICE_SELECTOR_LIST
-      // ? `https://${process.env.SERVICE_SELECTOR_LIST}`
-      'https://dg194mksjrqfz.cloudfront.net'
+    ? `http://${process.env.SERVICE_SELECTOR_LIST}`
+    : `https://${process.env.SERVICE_SELECTOR_LIST}`
 
 let registryIndex = 0
 
+export const getClusterUrl = (cluster?: string): string => {
+  if (!cluster) {
+    return DEV_CLUSTER
+  }
+  if (cluster === 'local') {
+    return 'http://localhost:7022'
+  }
+  if (cluster === 'localSpesh') {
+    return 'http://localhost:9022'
+  }
+  return cluster
+}
+
 export type ServiceUrl = {
@@ -41,2 +59,6 @@ protocol: string
 ): Promise<ServiceUrl> {
+  if (!cluster) {
+    throw new Error('No cluster url defined')
+  }
+
   try {
@@ -43,0 +65,0 @@ const registryUrls = await Promise.race([

New alerts

Environment variable access

Supply chain risk

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

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

72595

increased by3.26%

Lines of code

218

increased by21.79%

Worsened metrics

Number of low supply chain risk alerts

10

increased by100%

No dependency changes