@learnpack/learnpack 5.0.340 → 5.0.341

package/lib/commands/serve.js

@@ -733,7 +733,8 @@ class ServeCommand extends SessionCommand_1.default {
             credentials,
         });
         const bucket = bucketStorage.bucket(process.env.GCP_BUCKET_NAME || "learnpack-packages");
-        const host = process.env.HOST;
+        const rawHost = process.env.HOST || "";
+        const host = rawHost.trim().split(/\s+#/)[0].trim();
         if (!host) {
             console.log("ERROR: HOST is not set, please set the HOST environment variable");
             process.exit(1);
@@ -742,13 +743,24 @@ class ServeCommand extends SessionCommand_1.default {
             console.log("INFO: HOST is set to", host);
         }
         // Initialize Redis (official client)
-        const redisUrl = process.env.REDIS_URL || "redis://localhost:6379";
+        let redisUrl = (process.env.REDIS_URL || "redis://localhost:6379").trim();
+        // Strip surrounding quotes if present (e.g. from .env)
+        if ((redisUrl.startsWith('"') && redisUrl.endsWith('"')) ||
+            (redisUrl.startsWith("'") && redisUrl.endsWith("'"))) {
+            redisUrl = redisUrl.slice(1, -1);
+        }
+        const isPlaceholderRedisUrl = redisUrl.includes("your_production_password") ||
+            redisUrl.includes("your-redis-instance.cloud");
         if (process.env.NODE_ENV === "production" &&
-            redisUrl === "redis://localhost:6379") {
+            (redisUrl === "redis://localhost:6379" || isPlaceholderRedisUrl)) {
             console.error("❌ REDIS_URL not configured for production environment!");
             console.warn("⚠️  History features will be unavailable");
             this.redis = null;
         }
+        else if (isPlaceholderRedisUrl) {
+            console.warn("⚠️  REDIS_URL looks like a placeholder; skipping Redis. History features (undo/redo) will be unavailable");
+            this.redis = null;
+        }
         else {
             try {
                 const useTLS = redisUrl.startsWith("rediss://");
@@ -797,7 +809,8 @@ class ServeCommand extends SessionCommand_1.default {
         const app = express();
         const server = http.createServer(app);
         (0, creatorSocket_1.initSocketIO)(server);
-        const PORT = process.env.PORT || 3000;
+        const rawPort = process.env.PORT || "3000";
+        const PORT = parseInt(String(rawPort).trim().split(/\s+#/)[0].trim(), 10) || 3000;
         const distPath = path.resolve(__dirname, "../creatorDist");
         // Servir archivos estáticos
         // app.use(express.static(distPath))