@smpx/koa-request 0.3.0 → 0.4.0

package/Request.js

@@ -10,6 +10,8 @@ const enableBasicAuth = require('./basicAuth');
 const enableStaticPaths = require('./staticPaths');
 const enableBotBanning = require('./botBanning');
 
+const {setLogger, logError} = require('./logger');
+
 const uaParser = new UAParser();
 
 const ONE_HOUR = 3600 * 1000;
@@ -123,19 +125,25 @@ function getIntegerKey(key) {
 }
 
 function addQuery(url, query = {}) {
-	const uri = new URL(url, 'http://localhost');
-	if (typeof query === 'string') {
-		query = new URLSearchParams(query);
-		for (const [key, val] of query) {
-			uri.searchParams.set(key, val);
+	try {
+		const uri = new URL(url, 'http://localhost');
+		if (typeof query === 'string') {
+			query = new URLSearchParams(query);
+			for (const [key, val] of query) {
+				uri.searchParams.set(key, val);
+			}
 		}
-	}
-	else {
-		for (const [key, val] of Object.entries(query)) {
-			uri.searchParams.set(key, val);
+		else {
+			for (const [key, val] of Object.entries(query)) {
+				uri.searchParams.set(key, val);
+			}
 		}
+		return `${uri.pathname}${uri.search}`;
+	}
+	catch (e) {
+		logError(e);
+		return url;
 	}
-	return `${uri.pathname}${uri.search}`;
 }
 
 const isProduction = (process.env.NODE_ENV === 'production');
@@ -1235,7 +1243,20 @@ class Request {
 			};
 		}
 
-		const refererUri = new URL(referer);
+		let refererUri;
+		try {
+			refererUri = new URL(referer, 'http://localhost');
+		}
+		catch (e) {
+			logError(e);
+			return {
+				name: '',
+				source: 'direct',
+				medium: 'direct',
+				term: '',
+			};
+		}
+
 		let host = refererUri.hostname;
 		const baseDomain = this.baseDomain();
 
@@ -1288,16 +1309,22 @@ class Request {
 	 * sets UTM cookies from a predefined url
 	 */
 	setUTMCookieFromUrl(url) {
-		const uri = (url instanceof URL) ? url : new URL(url, 'http://localhost');
-		const params = uri.searchParams;
-		this.setUTMCookieFromQuery({
-			utm_source: params.get('utm_source'),
-			utm_medium: params.get('utm_medium'),
-			utm_campaign: params.get('utm_campaign'),
-			utm_term: params.get('utm_term'),
-			utm_content: params.get('utm_content'),
-			gclid: params.get('gclid'),
-		});
+		try {
+			const uri = (url instanceof URL) ? url : new URL(url, 'http://localhost');
+			const params = uri.searchParams;
+			this.setUTMCookieFromQuery({
+				utm_source: params.get('utm_source'),
+				utm_medium: params.get('utm_medium'),
+				utm_campaign: params.get('utm_campaign'),
+				utm_term: params.get('utm_term'),
+				utm_content: params.get('utm_content'),
+				gclid: params.get('gclid'),
+			});
+		}
+		catch (e) {
+			logError(e);
+			// ignore error
+		}
 	}
 
 	/**
@@ -1354,19 +1381,25 @@ class Request {
 	}
 
 	setAffidCookieFromUrl(url) {
-		const uri = (url instanceof URL) ? url : new URL(url, 'http://localhost');
-		const params = uri.searchParams;
-		const affid = params.get(AFFID_PARAM);
-		if (!affid) return;
-		const subaffid = params.get(SUBAFFID_PARAM);
-
-		this.cookie(
-			AFFID_COOKIE,
-			joinCookieParts([affid, subaffid]), {
-				maxAge: AFFID_COOKIE_DURATION,
-				domain: '*',
-			},
-		);
+		try {
+			const uri = (url instanceof URL) ? url : new URL(url, 'http://localhost');
+			const params = uri.searchParams;
+			const affid = params.get(AFFID_PARAM);
+			if (!affid) return;
+			const subaffid = params.get(SUBAFFID_PARAM);
+
+			this.cookie(
+				AFFID_COOKIE,
+				joinCookieParts([affid, subaffid]), {
+					maxAge: AFFID_COOKIE_DURATION,
+					domain: '*',
+				},
+			);
+		}
+		catch (e) {
+			logError(e);
+			// ignore error
+		}
 	}
 
 	handlePlatformModification() {
@@ -1496,7 +1529,7 @@ class Request {
 		}
 		catch (e) {
 			this._flash = '';
-			console.error('Error parsing flash message', e);
+			logError('Error parsing flash message', e);
 		}
 
 		this.cookie(FLASH_COOKIE, null);
@@ -1578,4 +1611,5 @@ class Request {
 	}
 }
 
+Request.setLogger = setLogger;
 module.exports = Request;

package/basicAuth.js

@@ -1,6 +1,6 @@
 const auth = require('koa-basic-auth');
 
-module.exports = function enableBasicAuth(app, options = {}) {
+module.exports = function enableBasicAuth(app, options) {
 	if (!options || options.enabled === false) return;
 
 	app.use(async (ctx, next) => {

package/botBanning.js

@@ -26,7 +26,7 @@ function banned(ctx, email) {
 	ctx.body = `<pre>Our system has detected unusual traffic from your ip ${ip}. Hence your ip has been banned temporarily.\n${emailStr}</pre>`;
 }
 
-module.exports = function enableBotBanning(app, options = {}) {
+module.exports = function enableBotBanning(app, options) {
 	if (!options || options.enabled === false) return;
 
 	let userAgents = [];

package/logger.js

@@ -0,0 +1,22 @@
+let globalLogger;
+
+/**
+ * set logger
+ */
+function setLogger(logger) {
+	globalLogger = logger;
+}
+
+function logError(...args) {
+	if (globalLogger !== undefined) {
+		if (globalLogger) globalLogger.error(...args);
+	}
+	else {
+		console.error(...args);
+	}
+}
+
+module.exports = {
+	setLogger,
+	logError,
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@smpx/koa-request",
-  "version": "0.3.0",
+  "version": "0.4.0",
   "description": "Handle basic tasks for koajs",
   "main": "index.js",
   "types": "index.d.ts",

package/rateLimit.js

@@ -1,6 +1,6 @@
 const {RateLimit} = require('koa2-ratelimit');
 
-module.exports = function enableRateLimit(app, options = {}) {
+module.exports = function enableRateLimit(app, options) {
 	if (!options || options.enabled === false) return;
 
 	const skip = options.skip;

package/staticPaths.js

@@ -20,8 +20,8 @@ function getMiddleware(options) {
 
 		if (options.path === '/') {
 			// root requires special handling to check if extension denotes a static path
-			const matches = ctx.path.match(/^\/(.*)\.(jpg|jpeg|gif|png|webp|avif|jxl|ico|css|js|mjs|json|ttf|otf|eot|woff|woff2|svg|svgz|xml|html|txt|ogg|ogv|mp4|av1|webm|rss|atom|zip|tgz|gz|rar|bz2|doc|xls|exe|ppt|tar|mid|midi|wav|bmp|rtf)$/);
-			if (!matches) {
+			const isStatic = /^\/(.*)\.(jpg|jpeg|gif|png|webp|avif|jxl|ico|css|js|mjs|json|ttf|otf|eot|woff|woff2|svg|svgz|xml|html|txt|ogg|ogv|mp4|av1|webm|rss|atom|zip|tgz|gz|rar|bz2|doc|xls|exe|ppt|tar|mid|midi|wav|bmp|rtf)$/.test(ctx.path);
+			if (!isStatic) {
 				await next();
 				return;
 			}
@@ -48,7 +48,7 @@ function getMiddleware(options) {
 
 		if (options.immutable) {
 			// return a 304 not modified response, as immutables can't be modified
-			if (ctx.headers['if-modified-since']) {
+			if (ctx.headers['if-modified-since'] && !ctx.response.get('Cache-Control')) {
 				ctx.status = 304;
 				return;
 			}