Last changes for production bug caught early

src/handlers/package_handler.js

@@ -812,7 +812,7 @@ async function getPackagesVersionTarball(req, res) {
   //let tarballURL = new url((pack.content.meta.tarball_url ? pack.content.meta.tarball_url : ""));
   let tarballURL = pack.content.meta.tarball_url;
 
-  if (tarballURL !== undefined) {
+  if (tarballURL !== undefined && false) { // todo the url feature doesn't work the way we would expect in production
     tarballURL = url.parse(pack.content.meta.tarball_url);
 
     if (
@@ -832,7 +832,7 @@ async function getPackagesVersionTarball(req, res) {
     tarballURL = tarballURL.toString();
   } else {
     // we are likely in a test environment. And will just leave this blank to error out on the client side.
-    tarballURL = "";
+    //tarballURL = "";
   }
 
   res.redirect(tarballURL);

src/utils.js

@@ -494,19 +494,19 @@ class StateStore {
         3,
         `StateStore.getState() Fail Return Anyway due to bug: ${ip}`
       );
-      return {
-        ok: true,
-        content: state,
-      };
+      //return {
+      //  ok: true,
+      //  content: state,
+      //};
       // The design here is flawed. Essentially it seems that because we are behind a load balancer
       // the process here needs to retain the stateStore, but there's no guarantee that
       // one request will hit the same state store, so we will temporarily disable this,
       // and instead use the state store as a microservice
-      //return {
-      //  ok: false,
-      //  short: "Not Found",
-      //  content: "Couldn't find IP within StateStore",
-      //};
+      return {
+        ok: false,
+        short: "Not Found",
+        content: "Couldn't find IP within StateStore",
+      };
     }
   }
   /**