maven-proxy 1.0.3 → 1.1.1

package/bin/maven-proxy.js

@@ -44,7 +44,8 @@ function resolveEffectiveMode(options) {
     return forced;
   }
 
-  return isProjectWorkspace(process.cwd()) ? "development" : "user";
+  // CLI defaults to user mode to load ~/maven-proxy/config unless explicitly overridden.
+  return "user";
 }
 
 function printHelp() {
@@ -213,9 +214,7 @@ async function startServer(options) {
 }
 
 function applyConfigOverrides(options) {
-  if (options.mode) {
-    process.env.MAVEN_PROXY_CONFIG_MODE = options.mode;
-  }
+  process.env.MAVEN_PROXY_CONFIG_MODE = resolveEffectiveMode(options);
 
   if (options.configPath) {
     process.env.MAVEN_PROXY_CONFIG_FILE = resolvePath(options.configPath);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "maven-proxy",
-  "version": "1.0.3",
+  "version": "1.1.1",
   "description": "Maven proxy with cache, HTTPS MITM for selected domains, and local repo publishing",
   "main": "src/index.js",
   "bin": {

package/src/cache/downloader.js

@@ -261,6 +261,27 @@ async function removeIfExists(filePath) {
   }
 }
 
+function toHost(urlObj) {
+  if (!urlObj || typeof urlObj !== "object") {
+    return "";
+  }
+
+  return String(urlObj.hostname || "");
+}
+
+function toBodyPreview(value, maxLength = 512) {
+  const text = String(value || "").replace(/\s+/g, " ").trim();
+  if (!text) {
+    return "";
+  }
+
+  if (text.length <= maxLength) {
+    return text;
+  }
+
+  return `${text.slice(0, maxLength)}...(truncated)`;
+}
+
 export class Downloader {
   constructor(config, domainMatcher, upstreamProxyManager = null) {
     this.config = config;
@@ -305,6 +326,8 @@ export class Downloader {
   }
 
   async #downloadAtomic(urlObj, finalPath, requestHeaders) {
+    const startedAt = Date.now();
+
     await fs.promises.mkdir(path.dirname(finalPath), { recursive: true });
     const tempPath = `${finalPath}.temp`;
     await removeIfExists(tempPath);
@@ -367,6 +390,14 @@ export class Downloader {
 
       await verifyFileSize(tempPath, metadata.contentLength);
       await fs.promises.rename(tempPath, finalPath);
+
+      const finalStats = await fs.promises.stat(finalPath);
+      this.logDownload("download succeeded", downloadUrl, {
+        host: hostname,
+        targetPath: finalPath,
+        size: finalStats.size,
+        elapsedMs: Date.now() - startedAt,
+      });
     } catch (error) {
       if (isLocalFsWriteError(error)) {
         if (!error.statusCode) {
@@ -381,6 +412,16 @@ export class Downloader {
         });
       }
 
+      this.logDownload("download failed", urlObj, {
+        host: toHost(urlObj),
+        code: error.code || "UNKNOWN",
+        statusCode: error.statusCode || 0,
+        targetPath: finalPath,
+        tempPath,
+        message: error.message,
+        upstreamBodyPreview: toBodyPreview(error.upstreamBody),
+      });
+
       await removeIfExists(tempPath);
       throw error;
     }

package/src/proxy/proxy-http-handler.js

@@ -74,6 +74,11 @@ function sendText(res, statusCode, message) {
   res.end(message);
 }
 
+function sendErrorText(res, statusCode, message, context = "proxy") {
+  console.error(`[${context}] response error status=${statusCode} message=${message}`);
+  sendText(res, statusCode, message);
+}
+
 function buildUrl(req, forcedProtocol = null) {
   const raw = req.url || "/";
   if (/^https?:\/\//i.test(raw)) {
@@ -143,7 +148,8 @@ function forwardDirectRequest(req, res, urlObj, timeoutMs, upstreamProxyManager
 
   upstreamReq.on("error", (error) => {
     if (!res.headersSent) {
-      sendText(res, 502, `Proxy forward failed: ${error.message}`);
+      const message = `Proxy forward failed: ${error.message}`;
+      sendErrorText(res, 502, message, "proxy");
     } else {
       res.destroy(error);
     }
@@ -158,7 +164,8 @@ export function createHttpRequestHandler({ config, downloader, upstreamProxyMana
     try {
       urlObj = buildUrl(req, forcedProtocol);
     } catch (error) {
-      sendText(res, 400, `Bad request: ${error.message}`);
+      const message = `Bad request: ${error.message}`;
+      sendErrorText(res, 400, message, "proxy");
       return;
     }
 
@@ -176,7 +183,8 @@ export function createHttpRequestHandler({ config, downloader, upstreamProxyMana
         includeHost: ecosystem !== "maven",
       });
     } catch (error) {
-      sendText(res, 400, `Invalid cache path: ${error.message}`);
+      const message = `Invalid cache path: ${error.message}`;
+      sendErrorText(res, 400, message, "proxy");
       return;
     }
 
@@ -210,7 +218,8 @@ export function createHttpRequestHandler({ config, downloader, upstreamProxyMana
 
       const statusCode = error.statusCode || 502;
       const label = statusCode === 500 ? "Local cache write failed" : "Download failed";
-      sendText(res, statusCode, `${label}: ${error.message}`);
+      const message = `${label}: ${error.message}`;
+      sendErrorText(res, statusCode, message, "proxy");
     }
   };
 }
@@ -218,7 +227,8 @@ export function createHttpRequestHandler({ config, downloader, upstreamProxyMana
 export function createMitmHttpServer(handleHttpRequestPath) {
   const server = http.createServer((req, res) => {
     handleHttpRequestPath(req, res, "https:").catch((error) => {
-      sendText(res, 500, `MITM request failed: ${error.message}`);
+      const message = `MITM request failed: ${error.message}`;
+      sendErrorText(res, 500, message, "proxy-mitm");
     });
   });
 

package/src/proxy/proxy-server.js

@@ -7,6 +7,11 @@ function sendText(res, statusCode, message) {
   res.end(message);
 }
 
+function sendErrorText(res, statusCode, message) {
+  console.error(`[proxy] response error status=${statusCode} message=${message}`);
+  sendText(res, statusCode, message);
+}
+
 export function startProxyServer(config, certManager, downloader, matchesDomain, upstreamProxyManager = null) {
   const handleHttpRequestPath = createHttpRequestHandler({
     config,
@@ -18,7 +23,8 @@ export function startProxyServer(config, certManager, downloader, matchesDomain,
 
   const server = http.createServer((req, res) => {
     handleHttpRequestPath(req, res, null).catch((error) => {
-      sendText(res, 500, `Proxy request failed: ${error.message}`);
+      const message = `Proxy request failed: ${error.message}`;
+      sendErrorText(res, 500, message);
     });
   });
 

package/src/proxy/upstream-proxy.js

@@ -118,7 +118,13 @@ export class UpstreamProxyManager {
 
     const cacheKey = `${proxyUrl}`;
     if (!this.agentCache.has(cacheKey)) {
-      this.agentCache.set(cacheKey, new ProxyAgent(proxyUrl));
+      // proxy-agent v6 expects resolver-style options for deterministic proxy routing.
+      this.agentCache.set(
+        cacheKey,
+        new ProxyAgent({
+          getProxyForUrl: () => proxyUrl,
+        }),
+      );
     }
 
     return this.agentCache.get(cacheKey);

package/src/repo/repo-server.js

@@ -110,8 +110,10 @@ export function startRepoServer(config, downloader = null) {
       fs.createReadStream(filePath).pipe(res);
     } catch (error) {
       const statusCode = error.statusCode && error.statusCode >= 400 ? 502 : 500;
+      const message = `Repo server error: ${error.message}`;
+      console.error(`[repo] response error status=${statusCode} message=${message}`);
       res.writeHead(statusCode, { "content-type": "text/plain; charset=utf-8" });
-      res.end(`Repo server error: ${error.message}`);
+      res.end(message);
     }
   });