fix: various

Author: Andrea Cosentino

src/models/plugin.model.ts

@@ -1882,7 +1882,20 @@ export type UniversalApiOptions = UniversalApiBaseOptions & (
 );
 
 /** @internal */
-export type UniversalApiOptionsRequired = Omit<Required<UniversalApiOptions>, "handlerMiddlewares" | "endpointPrefix"> & { endpointPrefix: string[], fullFsDir: string | null, config: ResolvedConfig, matcher: AntPathMatcher, middlewares: UniversalApiMiddleware[] };
+export type UniversalApiOptionsRequired = Omit<Required<UniversalApiOptions>, "handlerMiddlewares" | "endpointPrefix"> & {
+	endpointPrefix: string[];
+	/**
+	 * Prefixes that were provided by the user but rejected because they resolve
+	 * to the root path ("/") after normalisation. Stored so that runAsyncInit
+	 * can emit a targeted warning through the logger instead of the generic
+	 * "prefix empty or invalid" message.
+	 */
+	rejectedPrefixes: string[];
+	fullFsDir: string | null;
+	config: ResolvedConfig;
+	matcher: AntPathMatcher;
+	middlewares: UniversalApiMiddleware[];
+};
 
 /** @internal */
 export interface HandledRequestData {

src/plugin/index.ts

@@ -13,6 +13,7 @@ export function universalApiPlugin(opts?: UniversalApiOptions): Plugin {
 		delay: 0,
 		gatewayTimeout: 0,
 		endpointPrefix: [],
+		rejectedPrefixes: [],
 		enableWs: false,
 		enablePreview: true,
 		fsDir: null,
@@ -41,14 +42,26 @@ export function universalApiPlugin(opts?: UniversalApiOptions): Plugin {
 		asyncInitPromise = (async () => {
 			if (!options.fsDir || !(await Utils.files.isDirExists(options.fullFsDir!))) {
 				options.fullFsDir = null;
-				logger.info(`Directory with path ${options.fsDir} doesn't exist.`);
+				logger.debug(`Directory with path ${options.fsDir} doesn't exist.`);
+			}
+			if (options.rejectedPrefixes.length > 0) {
+				logger.warn(
+					`The following endpointPrefix values resolve to the root path and have been ignored: ` +
+					`[${options.rejectedPrefixes.map(p => `"${p || "/"}"`).join(", ")}]. ` +
+					`The root path "/" cannot be used as a prefix because it would intercept every request. ` +
+					`Use a more specific prefix such as "/api".`
+				);
 			}
 			if (options.endpointPrefix.length === 0) {
-				logger.warn(`Endpoint prefix empty or invalid`);
+				logger.warn(
+					options.rejectedPrefixes.length > 0
+						? `All configured prefixes were invalid (see warning above) — plugin disabled.`
+						: `endpointPrefix is empty or was not provided — plugin disabled.`
+				);
 				options.disable = true;
 			}
 
-			logger.info(`plugin ${options.disable ? "disabled" : "started"}`);
+			logger.debug(`plugin ${options.disable ? "disabled" : "started"}`);
 			logger.debug("Vite configResolved: FINISH");
 
 			/* v8 ignore start */
@@ -136,7 +149,7 @@ export function universalApiPlugin(opts?: UniversalApiOptions): Plugin {
 			options = Utils.plugin.initOptions(opts, conf);
 			logger = new Logger(Constants.PLUGIN_NAME, options.logLevel);
 			logger.debug("Vite configResolved: START");
-			logger.info(`plugin initializing ...`);
+			logger.debug(`plugin initializing ...`);
 		},
 		configureServer(server) {
 			// INFO Return a function so Vite defers its execution to after the built-in middlewares are registered

src/utils/plugin.ts

@@ -498,7 +498,7 @@ async function handlingApiRestRequest(logger: ILogger, matcher: AntPathMatcher,
 		}
 		if (handler !== null) {
 			logger.debug("handlingApiRestRequest: using REST api");
-			logger.info("Request handling with REST API: handler matched= ", handler.pattern);
+			logger.debug("Request handling with REST API: handler matched= ", handler.pattern);
 
 			if (handler.authenticate) {
 				try {
@@ -572,7 +572,7 @@ const runPluginInternal = async (req: IncomingMessage, res: ServerResponse, logg
 		}
 
 		if (!Utils.request.matchesEndpointPrefix(req.url, endpointPrefix)) {
-			logger.info(`runPluginInternal: Request with url ${req.url} doesn't match endpointPrefix option.`);
+			logger.debug(`runPluginInternal: Request with url ${req.url} doesn't match endpointPrefix option.`);
 			throw new UniversalApiError("Request doesn't match endpointPrefix", "NO_HANDLER", fullUrl.pathname);
 		}
 		const request = Utils.request.createRequest(req);

src/utils/utils.ts

@@ -64,17 +64,20 @@ export const Utils = {
 				endpoint = Utils.request.removeSlash(endpoint, "trailing");
 				return endpoint;
 			};
-			const endpointPrefix = opts?.endpointPrefix
+			const rawPrefixes: string[] = opts?.endpointPrefix
 				? Array.isArray(opts.endpointPrefix) && opts.endpointPrefix.length > 0
 					? opts.endpointPrefix.map(normalizePrefix)
 					: Array.isArray(opts.endpointPrefix) ? opts.endpointPrefix : [normalizePrefix(opts.endpointPrefix)]
 				: ['/api'];
+			const rejectedPrefixes = rawPrefixes.filter(el => el === "" || el === "/");
+			const endpointPrefix = rawPrefixes.filter(el => el !== "" && el !== "/");
 			return {
 				disable: opts?.disable ?? false,
 				logLevel: opts?.logLevel || "info",
 				delay: opts?.delay ? opts.delay : 0,
 				gatewayTimeout: opts?.gatewayTimeout ?? 0,
-				endpointPrefix: endpointPrefix.filter(el => el !== "" && el !== "/"),
+				rejectedPrefixes,
+				endpointPrefix,
 				enableWs: opts?.enableWs ?? false,
 				enablePreview: opts?.enablePreview ?? true,
 				fsDir: opts?.fsDir ?? null,
@@ -984,9 +987,8 @@ export const Utils = {
 			return result;
 		},
 		applyPaginationAndFilters(request: UniversalApiRequest<any>, paginationHandler: UniversalApiRestFsHandler["pagination"], filtersHandler: UniversalApiRestFsHandler["filters"], paginationPlugin: UniversalApiOptions["pagination"], filtersPlugin: UniversalApiOptions["filters"], dataFile: { originalData: any, data: any, mimeType: string, total: number }) {
-			const data = JSON.parse(dataFile.data);
-			dataFile.originalData = JSON.parse(dataFile.data);
-			dataFile.data = data;
+			dataFile.data = JSON.parse(dataFile.data);
+			dataFile.originalData = Utils.plugin.cloneData(dataFile.data);
 			const IS_ARRAY = Array.isArray(dataFile.data);
 			if (![null, undefined].includes(dataFile.data)) {
 				const { pagination, filters } = this.getPaginationAndFilters(request, paginationHandler, filtersHandler, paginationPlugin, filtersPlugin);