-> v0.11.1

Ignore files and directories with leading dots except .well-known

CHANGELOG.md

@@ -1,5 +1,10 @@
 # sapper changelog
 
+## 0.11.1
+
+* Limit routes with leading dots to `.well-known` URIs ([#252](https://github.com/sveltejs/sapper/issues/252))
+* Allow server routes to sit in front of pages ([#236](https://github.com/sveltejs/sapper/pull/236))
+
 ## 0.11.0
 
 * Create launcher file ([#240](https://github.com/sveltejs/sapper/issues/240))

package.json

@@ -1,6 +1,6 @@
 {
   "name": "sapper",
-  "version": "0.11.0",
+  "version": "0.11.1",
   "description": "Military-grade apps, engineered by Svelte",
   "main": "dist/middleware.ts.js",
   "bin": {
@@ -40,7 +40,6 @@
     "webpack-format-messages": "^1.0.2"
   },
   "devDependencies": {
-    "@std/esm": "^0.26.0",
     "@types/glob": "^5.0.34",
     "@types/mkdirp": "^0.5.2",
     "@types/rimraf": "^2.0.2",
@@ -48,7 +47,6 @@
     "eslint": "^4.13.1",
     "eslint-plugin-import": "^2.8.0",
     "express": "^4.16.3",
-    "get-port": "^3.2.0",
     "mocha": "^5.0.4",
     "nightmare": "^3.0.0",
     "npm-run-all": "^4.1.2",
@@ -61,7 +59,6 @@
     "serve-static": "^1.13.2",
     "svelte": "^2.4.4",
     "svelte-loader": "^2.9.0",
-    "ts-node": "^6.0.2",
     "typescript": "^2.8.3",
     "walk-sync": "^0.3.2",
     "webpack": "^4.1.0"

src/core/create_manifests.ts

@@ -45,11 +45,13 @@ function generate_client(routes: Route[], path_to_routes: string, dev_port?: num
 		export const routes = [
 			${routes
 				.map(route => {
-					if (route.type !== 'page') {
+					const page = route.handlers.find(({ type }) => type === 'page');
+
+					if (!page) {
 						return `{ pattern: ${route.pattern}, ignore: true }`;
 					}
 
-					const file = posixify(`${path_to_routes}/${route.file}`);
+					const file = posixify(`${path_to_routes}/${page.file}`);
 
 					if (route.id === '_4xx' || route.id === '_5xx') {
 						return `{ error: '${route.id.slice(1)}', load: () => import(/* webpackChunkName: "${route.id}" */ '${file}') }`;
@@ -85,28 +87,36 @@ function generate_server(routes: Route[], path_to_routes: string) {
 	let code = `
 		// This file is generated by Sapper — do not edit it!
 		${routes
-			.map(route => {
+			.map(route =>
-				const file = posixify(`${path_to_routes}/${route.file}`);
+				route.handlers
-				return route.type === 'page'
+					.map(({ type, file }, index) => {
-					? `import ${route.id} from '${file}';`
+						const module = posixify(`${path_to_routes}/${file}`);
-					: `import * as ${route.id} from '${file}';`;
+
-			})
+						return type === 'page'
+							? `import ${route.id}${index} from '${module}';`
+							: `import * as ${route.id}${index} from '${module}';`;
+					})
+					.join('\n')
+			)
 			.join('\n')}
 
 		export const routes = [
 			${routes
 				.map(route => {
-					const file = posixify(`../../${route.file}`);
+					const handlers = route.handlers
+						.map(({ type }, index) =>
+							`{ type: '${type}', module: ${route.id}${index} }`)
+						.join(', ');
 
 					if (route.id === '_4xx' || route.id === '_5xx') {
-						return `{ error: '${route.id.slice(1)}', module: ${route.id} }`;
+						return `{ error: '${route.id.slice(1)}', handlers: [${handlers}] }`;
 					}
 
 					const params = route.params.length === 0
 						? '{}'
 						: `{ ${route.params.map((part, i) => `${part}: match[${i + 1}]`).join(', ')} }`;
 
-					return `{ id: '${route.id}', type: '${route.type}', pattern: ${route.pattern}, params: ${route.params.length > 0 ? `match` : `()`} => (${params}), module: ${route.id} }`;
+					return `{ id: '${route.id}', pattern: ${route.pattern}, params: ${route.params.length > 0 ? `match` : `()`} => (${params}), handlers: [${handlers}] }`;
 				})
 				.join(',\n\t')
 			}

src/core/create_routes.ts

@@ -5,8 +5,9 @@ import { Route } from '../interfaces';
 
 export default function create_routes({ files } = { files: glob.sync('**/*.*', { cwd: locations.routes(), dot: true, nodir: true }) }) {
 	const routes: Route[] = files
+		.filter((file: string) => !/(^|\/|\\)_/.test(file))
 		.map((file: string) => {
-			if (/(^|\/|\\)_/.test(file)) return;
+			if (/(^|\/|\\)(_|\.(?!well-known))/.test(file)) return;
 
 			if (/]\[/.test(file)) {
 				throw new Error(`Invalid route ${file} — parameters must be separated`);
@@ -16,6 +17,60 @@ export default function create_routes({ files } = { files: glob.sync('**/*.*', {
 			const parts = base.split('/'); // glob output is always posix-style
 			if (parts[parts.length - 1] === 'index') parts.pop();
 
+			return {
+				files: [file],
+				base,
+				parts
+			};
+		})
+		.filter(Boolean)
+		.filter((a, index, array) => {
+			const found = array.slice(index + 1).find(b => a.base === b.base);
+			if (found) found.files.push(...a.files);
+			return !found;
+		})
+		.sort((a, b) => {
+			const max = Math.max(a.parts.length, b.parts.length);
+
+			if (max === 1) {
+				if (a.parts[0] === '4xx' || a.parts[0] === '5xx') return -1;
+				if (b.parts[0] === '4xx' || b.parts[0] === '5xx') return 1;
+			}
+
+			for (let i = 0; i < max; i += 1) {
+				const a_part = a.parts[i];
+				const b_part = b.parts[i];
+
+				if (!a_part) return -1;
+				if (!b_part) return 1;
+
+				const a_sub_parts = get_sub_parts(a_part);
+				const b_sub_parts = get_sub_parts(b_part);
+				const max = Math.max(a_sub_parts.length, b_sub_parts.length);
+
+				for (let i = 0; i < max; i += 1) {
+					const a_sub_part = a_sub_parts[i];
+					const b_sub_part = b_sub_parts[i];
+
+					if (!a_sub_part) return 1; // b is more specific, so goes first
+					if (!b_sub_part) return -1;
+
+					if (a_sub_part.dynamic !== b_sub_part.dynamic) {
+						return a_sub_part.dynamic ? 1 : -1;
+					}
+
+					if (!a_sub_part.dynamic && a_sub_part.content !== b_sub_part.content) {
+						return (
+							(b_sub_part.content.length - a_sub_part.content.length) ||
+							(a_sub_part.content < b_sub_part.content ? -1 : 1)
+						);
+					}
+				}
+			}
+
+			throw new Error(`The ${a.base} and ${b.base} routes clash`);
+		})
+		.map(({ files, base, parts }) => {
 			const id = (
 				parts.join('_').replace(/[[\]]/g, '$').replace(/^\d/, '_$&').replace(/[^a-zA-Z0-9_$]/g, '_')
 			 ) || '_';
@@ -63,54 +118,26 @@ export default function create_routes({ files } = { files: glob.sync('**/*.*', {
 
 			return {
 				id,
-				type: path.extname(file) === '.html' ? 'page' : 'route',
+				handlers: files.map(file => ({
-				file,
+					type: path.extname(file) === '.html' ? 'page' : 'route',
+					file
+				})).sort((a, b) => {
+					if (a.type === 'page' && b.type === 'route') {
+						return 1;
+					}
+
+					if (a.type === 'route' && b.type === 'page') {
+						return -1;
+					}
+
+					return 0;
+				}),
 				pattern,
 				test,
 				exec,
 				parts,
 				params
 			};
-		})
-		.filter(Boolean)
-		.sort((a: Route, b: Route) => {
-			if (a.file === '4xx.html' || a.file === '5xx.html') return -1;
-			if (b.file === '4xx.html' || b.file === '5xx.html') return 1;
-
-			const max = Math.max(a.parts.length, b.parts.length);
-
-			for (let i = 0; i < max; i += 1) {
-				const a_part = a.parts[i];
-				const b_part = b.parts[i];
-
-				if (!a_part) return -1;
-				if (!b_part) return 1;
-
-				const a_sub_parts = get_sub_parts(a_part);
-				const b_sub_parts = get_sub_parts(b_part);
-				const max = Math.max(a_sub_parts.length, b_sub_parts.length);
-
-				for (let i = 0; i < max; i += 1) {
-					const a_sub_part = a_sub_parts[i];
-					const b_sub_part = b_sub_parts[i];
-
-					if (!a_sub_part) return 1; // b is more specific, so goes first
-					if (!b_sub_part) return -1;
-
-					if (a_sub_part.dynamic !== b_sub_part.dynamic) {
-						return a_sub_part.dynamic ? 1 : -1;
-					}
-
-					if (!a_sub_part.dynamic && a_sub_part.content !== b_sub_part.content) {
-						return (
-							(b_sub_part.content.length - a_sub_part.content.length) ||
-							(a_sub_part.content < b_sub_part.content ? -1 : 1)
-						);
-					}
-				}
-			}
-
-			throw new Error(`The ${a.file} and ${b.file} routes clash`);
 		});
 
 	return routes;

src/interfaces.ts

@@ -1,7 +1,9 @@
 export type Route = {
 	id: string;
-	type: 'page' | 'route';
+	handlers: {
-	file: string;
+		type: 'page' | 'route';
+		file: string;
+	}[];
 	pattern: RegExp;
 	test: (url: string) => boolean;
 	exec: (url: string) => Record<string, string>;

src/middleware.ts

@@ -143,195 +143,212 @@ function get_route_handler(chunks: Record<string, string>, routes: RouteObject[]
 	function handle_route(route: RouteObject, req: Req, res: ServerResponse) {
 		req.params = route.params(route.pattern.exec(req.path));
 
-		const mod = route.module;
+		const handlers = route.handlers[Symbol.iterator]();
 
-		if (route.type === 'page') {
+		function next() {
-			res.setHeader('Content-Type', 'text/html');
+			try {
+				const { value: handler, done } = handlers.next();
 
-			// preload main.js and current route
+				if (done) {
-			// TODO detect other stuff we can preload? images, CSS, fonts?
+					handle_error(req, res, 404, 'Not found');
-			const link = []
+					return;
-				.concat(chunks.main, chunks[route.id])
+				}
-				.filter(file => !file.match(/\.map$/))
-				.map(file => `<${req.baseUrl}/client/${file}>;rel="preload";as="script"`)
-				.join(', ');
 
-			res.setHeader('Link', link);
+				const mod = handler.module;
 
-			const store = store_getter ? store_getter(req) : null;
+				if (handler.type === 'page') {
-			const data = { params: req.params, query: req.query };
+					res.setHeader('Content-Type', 'text/html');
 
-			let redirect: { statusCode: number, location: string };
+					// preload main.js and current route
-			let error: { statusCode: number, message: Error | string };
+					// TODO detect other stuff we can preload? images, CSS, fonts?
+					const link = []
+						.concat(chunks.main, chunks[route.id])
+						.filter(file => !file.match(/\.map$/))
+						.map(file => `<${req.baseUrl}/client/${file}>;rel="preload";as="script"`)
+						.join(', ');
 
-			Promise.resolve(
+					res.setHeader('Link', link);
-				mod.preload ? mod.preload.call({
-					redirect: (statusCode: number, location: string) => {
-						redirect = { statusCode, location };
-					},
-					error: (statusCode: number, message: Error | string) => {
-						error = { statusCode, message };
-					},
-					fetch: (url: string, opts?: any) => {
-						const parsed = new URL(url, `http://127.0.0.1:${process.env.PORT}${req.baseUrl ? req.baseUrl + '/'  :''}`);
 
-						if (opts) {
+					const store = store_getter ? store_getter(req) : null;
-							opts = Object.assign({}, opts);
+					const data = { params: req.params, query: req.query };
 
-							const include_cookies = (
+					let redirect: { statusCode: number, location: string };
-								opts.credentials === 'include' ||
+					let error: { statusCode: number, message: Error | string };
-								opts.credentials === 'same-origin' && parsed.origin === `http://127.0.0.1:${process.env.PORT}`
-							);
 
-							if (include_cookies) {
+					Promise.resolve(
-								const cookies: Record<string, string> = {};
+						mod.preload ? mod.preload.call({
-								if (!opts.headers) opts.headers = {};
+							redirect: (statusCode: number, location: string) => {
+								redirect = { statusCode, location };
+							},
+							error: (statusCode: number, message: Error | string) => {
+								error = { statusCode, message };
+							},
+							fetch: (url: string, opts?: any) => {
+								const parsed = new URL(url, `http://127.0.0.1:${process.env.PORT}${req.baseUrl ? req.baseUrl + '/'  :''}`);
 
-								const str = []
+								if (opts) {
-									.concat(
+									opts = Object.assign({}, opts);
-										cookie.parse(req.headers.cookie || ''),
-										cookie.parse(opts.headers.cookie || ''),
-										cookie.parse(res.getHeader('Set-Cookie') || '')
-									)
-									.map(cookie => {
-										return Object.keys(cookie)
-											.map(name => `${name}=${encodeURIComponent(cookie[name])}`)
-											.join('; ');
-									})
-									.filter(Boolean)
-									.join(', ');
 
-								opts.headers.cookie = str;
+									const include_cookies = (
-							}
+										opts.credentials === 'include' ||
+										opts.credentials === 'same-origin' && parsed.origin === `http://127.0.0.1:${process.env.PORT}`
+									);
+
+									if (include_cookies) {
+										const cookies: Record<string, string> = {};
+										if (!opts.headers) opts.headers = {};
+
+										const str = []
+											.concat(
+												cookie.parse(req.headers.cookie || ''),
+												cookie.parse(opts.headers.cookie || ''),
+												cookie.parse(res.getHeader('Set-Cookie') || '')
+											)
+											.map(cookie => {
+												return Object.keys(cookie)
+													.map(name => `${name}=${encodeURIComponent(cookie[name])}`)
+													.join('; ');
+											})
+											.filter(Boolean)
+											.join(', ');
+
+										opts.headers.cookie = str;
+									}
+								}
+
+								return fetch(parsed.href, opts);
+							},
+						store
+						}, req) : {}
+					).catch(err => {
+						error = { statusCode: 500, message: err };
+					}).then(preloaded => {
+						if (redirect) {
+							res.statusCode = redirect.statusCode;
+							res.setHeader('Location', `${req.baseUrl}/${redirect.location}`);
+							res.end();
+
+							return;
 						}
 
-						return fetch(parsed.href, opts);
+						if (error) {
-					},
+							handle_error(req, res, error.statusCode, error.message);
-					store
+							return;
-				}, req) : {}
+						}
-			).catch(err => {
-				error = { statusCode: 500, message: err };
-			}).then(preloaded => {
-				if (redirect) {
-					res.statusCode = redirect.statusCode;
-					res.setHeader('Location', `${req.baseUrl}/${redirect.location}`);
-					res.end();
 
-					return;
+						const serialized = {
-				}
+							preloaded: mod.preload && try_serialize(preloaded),
+							store: store && try_serialize(store.get())
+						};
+						Object.assign(data, preloaded);
 
-				if (error) {
+						const { html, head, css } = mod.render(data, {
-					handle_error(req, res, error.statusCode, error.message);
+							store
-					return;
+						});
-				}
 
-				const serialized = {
+						let scripts = []
-					preloaded: mod.preload && try_serialize(preloaded),
+							.concat(chunks.main) // chunks main might be an array. it might not! thanks, webpack
-					store: store && try_serialize(store.get())
+							.filter(file => !file.match(/\.map$/))
-				};
+							.map(file => `<script src='${req.baseUrl}/client/${file}'></script>`)
-				Object.assign(data, preloaded);
+							.join('');
 
-				const { html, head, css } = mod.render(data, {
+						let inline_script = `__SAPPER__={${[
-					store
+							`baseUrl: "${req.baseUrl}"`,
-				});
+							serialized.preloaded && `preloaded: ${serialized.preloaded}`,
+							serialized.store && `store: ${serialized.store}`
+						].filter(Boolean).join(',')}};`;
 
-				let scripts = []
+						const has_service_worker = fs.existsSync(path.join(locations.dest(), 'service-worker.js'));
-					.concat(chunks.main) // chunks main might be an array. it might not! thanks, webpack
+						if (has_service_worker) {
-					.filter(file => !file.match(/\.map$/))
+							inline_script += `if ('serviceWorker' in navigator) navigator.serviceWorker.register('${req.baseUrl}/service-worker.js');`;
-					.map(file => `<script src='${req.baseUrl}/client/${file}'></script>`)
+						}
-					.join('');
 
-				let inline_script = `__SAPPER__={${[
+						const page = template()
-					`baseUrl: "${req.baseUrl}"`,
+							.replace('%sapper.base%', `<base href="${req.baseUrl}/">`)
-					serialized.preloaded && `preloaded: ${serialized.preloaded}`,
+							.replace('%sapper.scripts%', `<script>${inline_script}</script>${scripts}`)
-					serialized.store && `store: ${serialized.store}`
+							.replace('%sapper.html%', html)
-				].filter(Boolean).join(',')}};`;
+							.replace('%sapper.head%', `<noscript id='sapper-head-start'></noscript>${head}<noscript id='sapper-head-end'></noscript>`)
+							.replace('%sapper.styles%', (css && css.code ? `<style>${css.code}</style>` : ''));
 
-				const has_service_worker = fs.existsSync(path.join(locations.dest(), 'service-worker.js'));
+						res.end(page);
-				if (has_service_worker) {
-					inline_script += `if ('serviceWorker' in navigator) navigator.serviceWorker.register('${req.baseUrl}/service-worker.js');`;
-				}
 
-				const page = template()
+						if (process.send) {
-					.replace('%sapper.base%', `<base href="${req.baseUrl}/">`)
+							process.send({
-					.replace('%sapper.scripts%', `<script>${inline_script}</script>${scripts}`)
+								__sapper__: true,
-					.replace('%sapper.html%', html)
+								url: req.url,
-					.replace('%sapper.head%', `<noscript id='sapper-head-start'></noscript>${head}<noscript id='sapper-head-end'></noscript>`)
+								method: req.method,
-					.replace('%sapper.styles%', (css && css.code ? `<style>${css.code}</style>` : ''));
+								status: 200,
-
+								type: 'text/html',
-				res.end(page);
+								body: page
-
+							});
-				if (process.send) {
+						}
-					process.send({
-						__sapper__: true,
-						url: req.url,
-						method: req.method,
-						status: 200,
-						type: 'text/html',
-						body: page
 					});
 				}
-			});
-		}
 
-		else {
+				else {
-			const method = req.method.toLowerCase();
+					const method = req.method.toLowerCase();
-			// 'delete' cannot be exported from a module because it is a keyword,
+					// 'delete' cannot be exported from a module because it is a keyword,
-			// so check for 'del' instead
+					// so check for 'del' instead
-			const method_export = method === 'delete' ? 'del' : method;
+					const method_export = method === 'delete' ? 'del' : method;
-			const handler = mod[method_export];
+					const handle_method = mod[method_export];
-			if (handler) {
+					if (handle_method) {
-				if (process.env.SAPPER_EXPORT) {
+						if (process.env.SAPPER_EXPORT) {
-					const { write, end, setHeader } = res;
+							const { write, end, setHeader } = res;
-					const chunks: any[] = [];
+							const chunks: any[] = [];
-					const headers: Record<string, string> = {};
+							const headers: Record<string, string> = {};
 
-					// intercept data so that it can be exported
+							// intercept data so that it can be exported
-					res.write = function(chunk: any) {
+							res.write = function(chunk: any) {
-						chunks.push(new Buffer(chunk));
+								chunks.push(new Buffer(chunk));
-						write.apply(res, arguments);
+								write.apply(res, arguments);
-					};
+							};
 
-					res.setHeader = function(name: string, value: string) {
+							res.setHeader = function(name: string, value: string) {
-						headers[name.toLowerCase()] = value;
+								headers[name.toLowerCase()] = value;
-						setHeader.apply(res, arguments);
+								setHeader.apply(res, arguments);
-					};
+							};
 
-					res.end = function(chunk?: any) {
+							res.end = function(chunk?: any) {
-						if (chunk) chunks.push(new Buffer(chunk));
+								if (chunk) chunks.push(new Buffer(chunk));
-						end.apply(res, arguments);
+								end.apply(res, arguments);
 
-						process.send({
+								process.send({
-							__sapper__: true,
+									__sapper__: true,
-							url: req.url,
+									url: req.url,
-							method: req.method,
+									method: req.method,
-							status: res.statusCode,
+									status: res.statusCode,
-							type: headers['content-type'],
+									type: headers['content-type'],
-							body: Buffer.concat(chunks).toString()
+									body: Buffer.concat(chunks).toString()
-						});
+								});
-					};
+							};
-				}
+						}
 
-				const handle_bad_result = (err?: Error) => {
+						const handle_bad_result = (err?: Error) => {
-					if (err) {
+							if (err) {
-						console.error(err.stack);
+								console.error(err.stack);
-						res.statusCode = 500;
+								res.statusCode = 500;
-						res.end(err.message);
+								res.end(err.message);
+							} else {
+								process.nextTick(next);
+							}
+						};
+
+						try {
+							handle_method(req, res, handle_bad_result);
+						} catch (err) {
+							handle_bad_result(err);
+						}
 					} else {
-						handle_error(req, res, 404, 'Not found');
+						// no matching handler for method
+						process.nextTick(next);
 					}
-				};
-
-				try {
-					handler(req, res, handle_bad_result);
-				} catch (err) {
-					handle_bad_result(err);
 				}
-			} else {
+			} catch (error) {
-				// no matching handler for method — 404
+				handle_error(req, res, 500, error);
-				handle_error(req, res, 404, 'Not found');
 			}
 		}
+
+		next();
 	}
 
 	const not_found_route = routes.find((route: RouteObject) => route.error === '4xx');
@@ -349,39 +366,62 @@ function get_route_handler(chunks: Record<string, string>, routes: RouteObject[]
 			? not_found_route
 			: error_route;
 
-		const title: string = not_found
+		function render_page({ head, css, html }) {
-			? 'Not found'
+			const page = template()
-			: `Internal server error: ${error.message}`;
+				.replace('%sapper.base%', `<base href="${req.baseUrl}/">`)
+				.replace('%sapper.scripts%', `<script>__SAPPER__={baseUrl: "${req.baseUrl}"}</script><script src='${req.baseUrl}/client/${chunks.main}'></script>`)
+				.replace('%sapper.html%', html)
+				.replace('%sapper.head%', `<noscript id='sapper-head-start'></noscript>${head}<noscript id='sapper-head-end'></noscript>`)
+				.replace('%sapper.styles%', (css && css.code ? `<style>${css.code}</style>` : ''));
 
-		const rendered = route ? route.module.render({
+			res.end(page);
-			status: statusCode,
+		}
-			error
-		}, {
-			store: store_getter && store_getter(req)
-		}) : { head: '', css: null, html: title };
 
-		const { head, css, html } = rendered;
+		function handle_notfound() {
+			const title: string = not_found
+				? 'Not found'
+				: `Internal server error: ${error.message}`;
 
-		const page = template()
+			render_page({ head: '', css: null, html: title });
-			.replace('%sapper.base%', `<base href="${req.baseUrl}/">`)
+		}
-			.replace('%sapper.scripts%', `<script>__SAPPER__={baseUrl: "${req.baseUrl}"}</script><script src='${req.baseUrl}/client/${chunks.main}'></script>`)
-			.replace('%sapper.html%', html)
-			.replace('%sapper.head%', `<noscript id='sapper-head-start'></noscript>${head}<noscript id='sapper-head-end'></noscript>`)
-			.replace('%sapper.styles%', (css && css.code ? `<style>${css.code}</style>` : ''));
 
-		res.end(page);
+		if (route) {
+			const handlers = route.handlers[Symbol.iterator]();
+
+			function next() {
+				const { value: handler, done } = handlers.next();
+
+				if (done) {
+					handle_notfound();
+				} else if (handler.type === 'page') {
+					render_page(handler.module.render({
+						status: statusCode,
+						error
+					}, {
+						store: store_getter && store_getter(req)
+					}));
+				} else {
+					const handle_method = mod[method_export];
+					if (handle_method) {
+						handle_method(req, res, next);
+					} else {
+						next();
+					}
+				}
+			}
+
+			next();
+		} else {
+			handle_notfound();
+		}
 	}
 
 	return function find_route(req: Req, res: ServerResponse) {
-		try {
+		for (const route of routes) {
-			for (const route of routes) {
+			if (!route.error && route.pattern.test(req.path)) return handle_route(route, req, res);
-				if (!route.error && route.pattern.test(req.path)) return handle_route(route, req, res);
-			}
-
-			handle_error(req, res, 404, 'Not found');
-		} catch (error) {
-			handle_error(req, res, 500, error);
 		}
+
+		handle_error(req, res, 404, 'Not found');
 	};
 }
 

test/unit/create_routes.test.js

@@ -2,7 +2,29 @@ const assert = require('assert');
 const { create_routes } = require('../../dist/core.ts.js');
 
 describe('create_routes', () => {
-	it('encodes caharcters not allowed in path', () => {
+	it('sorts handlers correctly', () => {
+		const routes = create_routes({
+			files: ['foo.html', 'foo.js']
+		});
+
+		assert.deepEqual(
+			routes.map(r => r.handlers),
+			[
+				[
+					{
+						type: 'route',
+						file: 'foo.js'
+					},
+					{
+						type: 'page',
+						file: 'foo.html'
+					}
+				]
+			]
+		)
+	});
+
+	it('encodes characters not allowed in path', () => {
 		const routes = create_routes({
 			files: [
 				'"',
@@ -27,7 +49,7 @@ describe('create_routes', () => {
 		});
 
 		assert.deepEqual(
-			routes.map(r => r.file),
+			routes.map(r => r.handlers[0].file),
 			[
 				'index.html',
 				'about.html',
@@ -59,7 +81,7 @@ describe('create_routes', () => {
 		});
 
 		assert.deepEqual(
-			routes.map(r => r.file),
+			routes.map(r => r.handlers[0].file),
 			[
 				'4xx.html',
 				'5xx.html',
@@ -95,7 +117,7 @@ describe('create_routes', () => {
 		});
 
 		assert.deepEqual(
-			routes.map(r => r.file),
+			routes.map(r => r.handlers[0].file),
 			[
 				'4xx.html',
 				'5xx.html',
@@ -125,7 +147,7 @@ describe('create_routes', () => {
 		for (let i = 0; i < routes.length; i += 1) {
 			const route = routes[i];
 			if (params = route.exec('/post/123')) {
-				file = route.file;
+				file = route.handlers[0].file;
 				break;
 			}
 		}
@@ -142,7 +164,7 @@ describe('create_routes', () => {
 		});
 
 		assert.deepEqual(
-			routes.map(r => r.file),
+			routes.map(r => r.handlers[0].file),
 			[
 				'index.html',
 				'e/f/g/h.html'
@@ -150,6 +172,17 @@ describe('create_routes', () => {
 		);
 	});
 
+	it('ignores files and directories with leading dots except .well-known', () => {
+		const routes = create_routes({
+			files: ['.well-known', '.unknown']
+		});
+
+		assert.deepEqual(
+			routes.map(r => r.handlers[0].file),
+			['.well-known']
+		);
+	});
+
 	it('matches /foo/:bar before /:baz/qux', () => {
 		const a = create_routes({
 			files: ['foo/[bar].html', '[baz]/qux.html']
@@ -159,12 +192,12 @@ describe('create_routes', () => {
 		});
 
 		assert.deepEqual(
-			a.map(r => r.file),
+			a.map(r => r.handlers[0].file),
 			['foo/[bar].html', '[baz]/qux.html']
 		);
 
 		assert.deepEqual(
-			b.map(r => r.file),
+			b.map(r => r.handlers[0].file),
 			['foo/[bar].html', '[baz]/qux.html']
 		);
 	});
@@ -174,13 +207,7 @@ describe('create_routes', () => {
 			create_routes({
 				files: ['[foo].html', '[bar]/index.html']
 			});
-		}, /The \[foo\].html and \[bar\]\/index.html routes clash/);
+		}, /The \[foo\] and \[bar\]\/index routes clash/);
-
-		assert.throws(() => {
-			create_routes({
-				files: ['foo.html', 'foo.js']
-			});
-		}, /The foo.html and foo.js routes clash/);
 	});
 
 	it('matches nested routes', () => {
@@ -203,7 +230,7 @@ describe('create_routes', () => {
 		});
 
 		assert.deepEqual(
-			routes.map(r => r.file),
+			routes.map(r => r.handlers[0].file),
 			['settings.html', 'settings/[submenu].html']
 		);
 	});