@sap-ux/ui5-proxy-middleware 1.5.12 → 1.6.0

package/dist/base/proxy.d.ts

@@ -1,12 +1,15 @@
 import type { Filter, Options } from 'http-proxy-middleware';
+import type { IncomingMessage, ServerResponse } from 'http';
 import type { ProxyConfig } from './types';
+import { ToolsLogger } from '@sap-ux/logger';
 /**
  * Function for proxying UI5 sources.
  *
  * @param config - proxy configuration
  * @param options - additional configuration options
  * @param filter - custom filter function which will be applied to all requests
+ * @param logger - optional logger instance
  * @returns Proxy function to use
  */
-export declare const ui5Proxy: (config: ProxyConfig, options?: Options, filter?: Filter) => import("http-proxy-middleware").RequestHandler;
+export declare const ui5Proxy: (config: ProxyConfig, options?: Options, filter?: Filter, logger?: ToolsLogger) => import("http-proxy-middleware").RequestHandler<IncomingMessage, ServerResponse<IncomingMessage>, (err?: any) => void>;
 //# sourceMappingURL=proxy.d.ts.map

package/dist/base/proxy.js

@@ -12,28 +12,38 @@ const https_proxy_agent_1 = require("https-proxy-agent");
  * @param config - proxy configuration
  * @param options - additional configuration options
  * @param filter - custom filter function which will be applied to all requests
+ * @param logger - optional logger instance
  * @returns Proxy function to use
  */
-const ui5Proxy = (config, options, filter) => {
-    const logger = new logger_1.ToolsLogger({
-        transports: [new logger_1.UI5ToolingTransport({ moduleName: 'ui5-proxy-middleware' })]
-    });
+const ui5Proxy = (config, options, filter, logger = new logger_1.ToolsLogger({
+    transports: [new logger_1.UI5ToolingTransport({ moduleName: 'ui5-proxy-middleware' })]
+})) => {
     const today = new Date();
     const etag = `W/"${config.version || 'ui5-latest-' + today.getDate() + today.getMonth() + today.getFullYear()}"`;
     const ui5Ver = config.version ? `/${config.version}` : '';
+    let proxyFilter = utils_1.filterCompressedHtmlFiles;
+    if (filter) {
+        proxyFilter = filter;
+    }
     const proxyConfig = {
+        on: {
+            proxyReq: (proxyReq, _req, res) => {
+                (0, utils_1.proxyRequestHandler)(proxyReq, res, etag);
+            },
+            proxyRes: (proxyRes) => {
+                (0, utils_1.proxyResponseHandler)(proxyRes, etag);
+            },
+            error: (err, req, res, target) => {
+                (0, utils_1.proxyErrorHandler)(err, req, logger, res, target);
+            }
+        },
         target: config.url,
         changeOrigin: true,
-        onProxyReq: (proxyReq, _req, res) => {
-            (0, utils_1.proxyRequestHandler)(proxyReq, res, etag, logger);
+        pathRewrite: (path, _req) => {
+            return path.startsWith(config.path) ? ui5Ver + path : ui5Ver + config.path + path;
         },
-        pathRewrite: { [config.path]: ui5Ver + config.path },
-        onProxyRes: (proxyRes) => {
-            (0, utils_1.proxyResponseHandler)(proxyRes, etag);
-        },
-        onError: (err, req, res, target) => {
-            (0, utils_1.proxyErrorHandler)(err, req, logger, res, target);
-        }
+        pathFilter: proxyFilter,
+        ...options
     };
     // update proxy config with values coming from args or ui5.yaml
     (0, utils_1.updateProxyEnv)(config.proxy);
@@ -41,12 +51,7 @@ const ui5Proxy = (config, options, filter) => {
     if (corporateProxy) {
         proxyConfig.agent = new https_proxy_agent_1.HttpsProxyAgent(corporateProxy);
     }
-    Object.assign(proxyConfig, options);
-    let proxyFilter = utils_1.filterCompressedHtmlFiles;
-    if (filter) {
-        proxyFilter = filter;
-    }
-    return (0, http_proxy_middleware_1.createProxyMiddleware)(proxyFilter, proxyConfig);
+    return (0, http_proxy_middleware_1.createProxyMiddleware)(proxyConfig);
 };
 exports.ui5Proxy = ui5Proxy;
 //# sourceMappingURL=proxy.js.map

package/dist/base/utils.d.ts

@@ -6,6 +6,7 @@ import type http from 'http';
 import type { ProxyConfig } from './types';
 import type { Url } from 'url';
 import type { ReaderCollection } from '@ui5/fs';
+import type { Socket } from 'node:net';
 /**
  * Handler for the proxy response event.
  * Sets an Etag which will be used for re-validation of the cached UI5 sources.
@@ -22,9 +23,8 @@ export declare const proxyResponseHandler: (proxyRes: IncomingMessage, etag: str
  * @param proxyReq - proxy request object
  * @param res - server response object
  * @param etag - Etag of the cached UI5 sources, normally the UI5 version
- * @param logger - Logger for loging the requests
  */
-export declare const proxyRequestHandler: (proxyReq: ClientRequest, res: ServerResponse, etag: string, logger: ToolsLogger) => void;
+export declare const proxyRequestHandler: (proxyReq: ClientRequest, res: ServerResponse, etag: string) => void;
 /**
  * Get user's proxy configuration.
  *
@@ -94,8 +94,9 @@ export declare function injectUI5Url(originalHtml: string, ui5Configs: ProxyConf
  * @param next - the next function, used to forward the request to the next available handler
  * @param ui5Configs - the UI5 configuration of the ui5-proxy-middleware
  * @param rootProject - the root project
+ * @param logger - logger to be used
  */
-export declare const injectScripts: (req: Request, res: Response, next: NextFunction, ui5Configs: ProxyConfig[], rootProject: ReaderCollection) => Promise<void>;
+export declare const injectScripts: (req: Request, res: Response, next: NextFunction, ui5Configs: ProxyConfig[], rootProject: ReaderCollection, logger?: ToolsLogger) => Promise<void>;
 /**
  * Filters comressed html files from UI5 CDN.
  * Avoid ERR_CONTENT_DECODING_FAILED on http request for gzip'd html files.
@@ -120,7 +121,7 @@ export declare function proxyErrorHandler(err: Error & {
 }, req: IncomingMessage & {
     next?: Function;
     originalUrl?: string;
-}, logger: ToolsLogger, _res?: ServerResponse, _target?: string | Partial<Url>): void;
+}, logger: ToolsLogger, _res?: ServerResponse | Socket, _target?: string | Partial<Url>): void;
 /**
  * Adjust UI5 bootstrap URLs to load directly from UI5 CDN.
  *

package/dist/base/utils.js

@@ -29,10 +29,8 @@ exports.proxyResponseHandler = proxyResponseHandler;
  * @param proxyReq - proxy request object
  * @param res - server response object
  * @param etag - Etag of the cached UI5 sources, normally the UI5 version
- * @param logger - Logger for loging the requests
  */
-const proxyRequestHandler = (proxyReq, res, etag, logger) => {
-    logger.debug(proxyReq.path);
+const proxyRequestHandler = (proxyReq, res, etag) => {
     if (proxyReq.getHeader('if-none-match') === etag) {
         res.statusCode = 304;
         res.end();
@@ -232,12 +230,14 @@ function injectUI5Url(originalHtml, ui5Configs) {
  * @param next - the next function, used to forward the request to the next available handler
  * @param ui5Configs - the UI5 configuration of the ui5-proxy-middleware
  * @param rootProject - the root project
+ * @param logger - logger to be used
  */
-const injectScripts = async (req, res, next, ui5Configs, rootProject) => {
+const injectScripts = async (req, res, next, ui5Configs, rootProject, logger) => {
     try {
         const htmlFileName = (0, exports.getHtmlFile)(req.url);
         const files = await rootProject.byGlob(`**/${htmlFileName}`);
         if (files.length === 0) {
+            logger?.warn('No HTML file found for direct load injection.');
             next();
         }
         else {
@@ -302,7 +302,7 @@ function proxyErrorHandler(err, req, logger, _res, _target) {
 function directLoadProxy(ui5Configs, rootProject, logger) {
     return async (req, res, next) => {
         try {
-            await (0, exports.injectScripts)(req, res, next, ui5Configs, rootProject);
+            await (0, exports.injectScripts)(req, res, next, ui5Configs, rootProject, logger);
         }
         catch (error) {
             logger.error(error);

package/dist/ui5/middleware.js

@@ -17,8 +17,7 @@ const dotenv_1 = __importDefault(require("dotenv"));
 function createProxyOptions(logger, config) {
     return {
         secure: config.secure !== undefined ? !!config.secure : true,
-        logLevel: config.debug ? 'debug' : 'info',
-        logProvider: () => logger
+        logger: config.debug ? logger : undefined
     };
 }
 /**
@@ -51,6 +50,7 @@ async function loadManifest(rootProject) {
 }
 module.exports = async ({ resources, options }) => {
     const logger = new logger_1.ToolsLogger({
+        logLevel: options.configuration?.debug ? logger_1.LogLevel.Debug : logger_1.LogLevel.Info,
         transports: [new logger_1.UI5ToolingTransport({ moduleName: 'ui5-proxy-middleware' })]
     });
     dotenv_1.default.config();
@@ -76,7 +76,7 @@ module.exports = async ({ resources, options }) => {
     // hide user and pass from proxy configuration for displaying it in the terminal
     const proxyInfo = (0, base_1.hideProxyCredentials)(corporateProxyServer);
     const proxyOptions = createProxyOptions(logger, config);
-    logger.info(`Starting ui5-proxy-middleware using following configuration:\nproxy: '${proxyInfo}'\nsecure: '${proxyOptions.secure}'\nlog: '${proxyOptions.logLevel}''\ndirectLoad: '${directLoad}'`);
+    logger.info(`Starting ui5-proxy-middleware using following configuration:\nproxy: '${proxyInfo}'\nsecure: '${proxyOptions.secure}'\nlog: '${config.debug ? 'debug' : 'info'}' \ndirectLoad: '${directLoad}'`);
     const configs = Array.isArray(config.ui5) ? config.ui5 : [config.ui5];
     const ui5Configs = [];
     const routes = [];
@@ -89,7 +89,7 @@ module.exports = async ({ resources, options }) => {
                 version: ui5Version,
                 proxy: config.proxy
             };
-            routes.push({ route: ui5Config.path, handler: (0, base_1.ui5Proxy)(ui5Config, proxyOptions) });
+            routes.push({ route: ui5Config.path, handler: (0, base_1.ui5Proxy)(ui5Config, proxyOptions, undefined, logger) });
             ui5Configs.push(ui5Config);
         }
     }

package/package.json

@@ -9,7 +9,7 @@
   "bugs": {
     "url": "https://github.com/SAP/open-ux-tools/issues?q=is%3Aopen+is%3Aissue+label%3Abug+label%3Aui5-proxy-middleware"
   },
-  "version": "1.5.12",
+  "version": "1.6.0",
   "license": "Apache-2.0",
   "author": "@SAP/ux-tools-team",
   "main": "dist/index.js",
@@ -22,7 +22,7 @@
   ],
   "dependencies": {
     "dotenv": "16.3.1",
-    "http-proxy-middleware": "2.0.9",
+    "http-proxy-middleware": "3.0.5",
     "https-proxy-agent": "5.0.1",
     "i18next": "25.3.0",
     "proxy-from-env": "1.1.0",