@siddharatha/adapter-node-rolldown 1.0.9 → 1.1.4

package/files/env.js

@@ -1,50 +1,94 @@
+import process from 'node:process';
+
+/* global ENV_PREFIX */
+
+const expected = new Set([
+    'SOCKET_PATH',
+    'HOST',
+    'PORT',
+    'ORIGIN',
+    'XFF_DEPTH',
+    'ADDRESS_HEADER',
+    'PROTOCOL_HEADER',
+    'HOST_HEADER',
+    'PORT_HEADER',
+    'BODY_SIZE_LIMIT',
+    'SHUTDOWN_TIMEOUT',
+    'IDLE_TIMEOUT',
+    'KEEP_ALIVE_TIMEOUT',
+    'HEADERS_TIMEOUT'
+]);
+
+const expected_unprefixed = new Set(['LISTEN_PID', 'LISTEN_FDS']);
+
+if (ENV_PREFIX) {
+    for (const name in process.env) {
+        if (name.startsWith(ENV_PREFIX)) {
+            const unprefixed = name.slice(ENV_PREFIX.length);
+            if (!expected.has(unprefixed)) {
+                throw new Error(
+                    `You should change envPrefix (${ENV_PREFIX}) to avoid conflicts with existing environment variables — unexpectedly saw ${name}`
+                );
+            }
+        }
+    }
+}
+
+/**
+ * @param {string} name
+ * @param {any} fallback
+ */
+function env(name, fallback) {
+    const prefix = expected_unprefixed.has(name) ? '' : ENV_PREFIX;
+    const prefixed = prefix + name;
+    return prefixed in process.env ? process.env[prefixed] : fallback;
+}
+
+const integer_regexp = /^\d+$/;
+
 /**
- * Environment configuration with sensible defaults
+ * Throw a consistently-structured parsing error for environment variables.
+ * @param {string} name
+ * @param {any} value
+ * @param {string} description
+ * @returns {never}
  */
-export const config = {
-    // Server configuration
-    port: parseInt(process.env.PORT || '3000', 10),
-    host: process.env.HOST || '0.0.0.0', // Important: use 0.0.0.0 for containers
-
-    // Performance options
-    compression: JSON.parse('COMPRESSION_ENABLED'),
-    compressionLevel: JSON.parse('COMPRESSION_LEVEL'),
-    bodyLimit: JSON.parse('BODY_LIMIT'),
-
-    // WebSocket configuration
-    websocket: {
-        enabled: JSON.parse('WEBSOCKET_ENABLED'),
-        path: JSON.parse('WEBSOCKET_PATH')
-    },
-
-    // OpenTelemetry configuration
-    telemetry: {
-        enabled: JSON.parse('TELEMETRY_ENABLED') && process.env.OTEL_ENABLED !== 'false',
-        serviceName: process.env.OTEL_SERVICE_NAME || 'sveltekit-app',
-        serviceVersion: process.env.OTEL_SERVICE_VERSION || '1.0.0',
-        endpoint: process.env.OTEL_EXPORTER_OTLP_ENDPOINT,
-        protocol: process.env.OTEL_EXPORTER_OTLP_PROTOCOL || 'http', // 'http' or 'grpc'
-        sampleRate: parseFloat(process.env.OTEL_SAMPLE_RATE || JSON.parse('TELEMETRY_SAMPLE_RATE')),
-        dynatraceToken: process.env.DYNATRACE_API_TOKEN,
-        customConfig: JSON.parse('TELEMETRY_CONFIG')
-    },
-
-    // Health check configuration
-    healthCheck: {
-        enabled: JSON.parse('HEALTH_CHECK_ENABLED')
-    },
-
-    // Graceful shutdown
-    gracefulShutdownTimeout: JSON.parse('GRACEFUL_SHUTDOWN_TIMEOUT'),
-
-    // Server tuning
-    keepAliveTimeout: parseInt(process.env.KEEP_ALIVE_TIMEOUT || '65000', 10),
-    headersTimeout: parseInt(process.env.HEADERS_TIMEOUT || '66000', 10),
-    maxRequestsPerSocket: parseInt(process.env.MAX_REQUESTS_PER_SOCKET || '0', 10),
-
-    // Origin for CORS (if needed)
-    origin: process.env.ORIGIN || undefined,
-
-    // XFF header handling (trust proxy)
-    trustProxy: process.env.TRUST_PROXY === 'true'
-};
+function parsing_error(name, value, description) {
+    throw new Error(
+        `Invalid value for environment variable ${name}: ${JSON.stringify(value)} (${description})`
+    );
+}
+
+/**
+ * Check the environment for a timeout value (non-negative integer) in seconds.
+ * @param {string} name
+ * @param {number} [fallback]
+ * @returns {number | undefined}
+ */
+function timeout_env(name, fallback) {
+    const raw = env(name, fallback);
+    if (!raw) {
+        return fallback;
+    }
+
+    if (!integer_regexp.test(raw)) {
+        parsing_error(name, raw, 'should be a non-negative integer');
+    }
+
+    const parsed = Number.parseInt(raw, 10);
+
+    // We don't technically need to check `Number.isNaN` because the value already passed the regexp test.
+    // However, just in case there's some new codepath introduced somewhere down the line, it's probably good
+    // to stick this in here.
+    if (Number.isNaN(parsed)) {
+        parsing_error(name, raw, 'should be a non-negative integer');
+    }
+
+    if (parsed < 0) {
+        parsing_error(name, raw, 'should be a non-negative integer');
+    }
+
+    return parsed;
+}
+
+export { env, timeout_env };