mirror of
https://github.com/kevin-DL/sapper.git
synced 2026-01-16 04:44:35 +00:00
Compare commits
6 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
d1fcd07c92 | ||
|
|
47a6d6f662 | ||
|
|
4b2b6440d0 | ||
|
|
566addd406 | ||
|
|
3d77dacbd6 | ||
|
|
51b4f9cbbf |
@@ -1,5 +1,9 @@
|
|||||||
# sapper changelog
|
# sapper changelog
|
||||||
|
|
||||||
|
## 0.15.4
|
||||||
|
|
||||||
|
* Add `ignore` option ([#326](https://github.com/sveltejs/sapper/pull/326))
|
||||||
|
|
||||||
## 0.15.3
|
## 0.15.3
|
||||||
|
|
||||||
* Crawl pages in parallel when exporting ([#329](https://github.com/sveltejs/sapper/pull/329))
|
* Crawl pages in parallel when exporting ([#329](https://github.com/sveltejs/sapper/pull/329))
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "sapper",
|
"name": "sapper",
|
||||||
"version": "0.15.3",
|
"version": "0.15.4",
|
||||||
"description": "Military-grade apps, engineered by Svelte",
|
"description": "Military-grade apps, engineered by Svelte",
|
||||||
"main": "dist/middleware.ts.js",
|
"main": "dist/middleware.ts.js",
|
||||||
"bin": {
|
"bin": {
|
||||||
|
|||||||
@@ -73,9 +73,18 @@ interface Component {
|
|||||||
preload: (data: any) => any | Promise<any>
|
preload: (data: any) => any | Promise<any>
|
||||||
}
|
}
|
||||||
|
|
||||||
|
const IGNORE = '__SAPPER__IGNORE__';
|
||||||
|
function toIgnore(uri: string, val: any) {
|
||||||
|
if (Array.isArray(val)) return val.some(x => toIgnore(uri, x));
|
||||||
|
if (val instanceof RegExp) return val.test(uri);
|
||||||
|
if (typeof val === 'function') return val(uri);
|
||||||
|
return uri.startsWith(val.charCodeAt(0) === 47 ? val : `/${val}`);
|
||||||
|
}
|
||||||
|
|
||||||
export default function middleware(opts: {
|
export default function middleware(opts: {
|
||||||
manifest: Manifest,
|
manifest: Manifest,
|
||||||
store: (req: Req) => Store,
|
store: (req: Req) => Store,
|
||||||
|
ignore?: any,
|
||||||
routes?: any // legacy
|
routes?: any // legacy
|
||||||
}) {
|
}) {
|
||||||
if (opts.routes) {
|
if (opts.routes) {
|
||||||
@@ -84,12 +93,19 @@ export default function middleware(opts: {
|
|||||||
|
|
||||||
const output = locations.dest();
|
const output = locations.dest();
|
||||||
|
|
||||||
const { manifest, store } = opts;
|
const { manifest, store, ignore } = opts;
|
||||||
|
|
||||||
let emitted_basepath = false;
|
let emitted_basepath = false;
|
||||||
|
|
||||||
const middleware = compose_handlers([
|
const middleware = compose_handlers([
|
||||||
|
ignore && ((req: Req, res: ServerResponse, next: () => void) => {
|
||||||
|
req[IGNORE] = toIgnore(req.path, ignore);
|
||||||
|
next();
|
||||||
|
}),
|
||||||
|
|
||||||
(req: Req, res: ServerResponse, next: () => void) => {
|
(req: Req, res: ServerResponse, next: () => void) => {
|
||||||
|
if (req[IGNORE]) return next();
|
||||||
|
|
||||||
if (req.baseUrl === undefined) {
|
if (req.baseUrl === undefined) {
|
||||||
let { originalUrl } = req;
|
let { originalUrl } = req;
|
||||||
if (req.url === '/' && originalUrl[originalUrl.length - 1] !== '/') {
|
if (req.url === '/' && originalUrl[originalUrl.length - 1] !== '/') {
|
||||||
@@ -163,6 +179,8 @@ function serve({ prefix, pathname, cache_control }: {
|
|||||||
: (file: string) => (cache.has(file) ? cache : cache.set(file, fs.readFileSync(path.resolve(output, file)))).get(file)
|
: (file: string) => (cache.has(file) ? cache : cache.set(file, fs.readFileSync(path.resolve(output, file)))).get(file)
|
||||||
|
|
||||||
return (req: Req, res: ServerResponse, next: () => void) => {
|
return (req: Req, res: ServerResponse, next: () => void) => {
|
||||||
|
if (req[IGNORE]) return next();
|
||||||
|
|
||||||
if (filter(req)) {
|
if (filter(req)) {
|
||||||
const type = lookup(req.path);
|
const type = lookup(req.path);
|
||||||
|
|
||||||
@@ -245,6 +263,8 @@ function get_server_route_handler(routes: ServerRoute[]) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
return function find_route(req: Req, res: ServerResponse, next: () => void) {
|
return function find_route(req: Req, res: ServerResponse, next: () => void) {
|
||||||
|
if (req[IGNORE]) return next();
|
||||||
|
|
||||||
for (const route of routes) {
|
for (const route of routes) {
|
||||||
if (route.pattern.test(req.path)) {
|
if (route.pattern.test(req.path)) {
|
||||||
handle_route(route, req, res, next);
|
handle_route(route, req, res, next);
|
||||||
@@ -494,7 +514,9 @@ function get_page_handler(manifest: Manifest, store_getter: (req: Req) => Store)
|
|||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
return function find_route(req: Req, res: ServerResponse) {
|
return function find_route(req: Req, res: ServerResponse, next: () => void) {
|
||||||
|
if (req[IGNORE]) return next();
|
||||||
|
|
||||||
if (!server_routes.some(route => route.pattern.test(req.path))) {
|
if (!server_routes.some(route => route.pattern.test(req.path))) {
|
||||||
for (const page of pages) {
|
for (const page of pages) {
|
||||||
if (page.pattern.test(req.path)) {
|
if (page.pattern.test(req.path)) {
|
||||||
|
|||||||
@@ -91,8 +91,14 @@ const middlewares = [
|
|||||||
return new Store({
|
return new Store({
|
||||||
title: 'Stored title'
|
title: 'Stored title'
|
||||||
});
|
});
|
||||||
}
|
},
|
||||||
})
|
ignore: [
|
||||||
|
/foobar/i,
|
||||||
|
'/buzz',
|
||||||
|
'fizz',
|
||||||
|
x => x === '/hello'
|
||||||
|
]
|
||||||
|
}),
|
||||||
];
|
];
|
||||||
|
|
||||||
if (BASEPATH) {
|
if (BASEPATH) {
|
||||||
@@ -101,4 +107,8 @@ if (BASEPATH) {
|
|||||||
app.use(...middlewares);
|
app.use(...middlewares);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
['foobar', 'buzz', 'fizzer', 'hello'].forEach(uri => {
|
||||||
|
app.get('/'+uri, (req, res) => res.end(uri));
|
||||||
|
});
|
||||||
|
|
||||||
app.listen(PORT);
|
app.listen(PORT);
|
||||||
@@ -477,6 +477,42 @@ function run({ mode, basepath = '' }) {
|
|||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
|
// Ignores are meant for top-level escape.
|
||||||
|
// ~> Sapper **should** own the entire {basepath} when designated.
|
||||||
|
if (!basepath) {
|
||||||
|
it('respects `options.ignore` values (RegExp)', () => {
|
||||||
|
return nightmare.goto(`${base}/foobar`)
|
||||||
|
.evaluate(() => document.documentElement.textContent)
|
||||||
|
.then(text => {
|
||||||
|
assert.equal(text, 'foobar');
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
it('respects `options.ignore` values (String #1)', () => {
|
||||||
|
return nightmare.goto(`${base}/buzz`)
|
||||||
|
.evaluate(() => document.documentElement.textContent)
|
||||||
|
.then(text => {
|
||||||
|
assert.equal(text, 'buzz');
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
it('respects `options.ignore` values (String #2)', () => {
|
||||||
|
return nightmare.goto(`${base}/fizzer`)
|
||||||
|
.evaluate(() => document.documentElement.textContent)
|
||||||
|
.then(text => {
|
||||||
|
assert.equal(text, 'fizzer');
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
it('respects `options.ignore` values (Function)', () => {
|
||||||
|
return nightmare.goto(`${base}/hello`)
|
||||||
|
.evaluate(() => document.documentElement.textContent)
|
||||||
|
.then(text => {
|
||||||
|
assert.equal(text, 'hello');
|
||||||
|
});
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
it('does not attempt client-side navigation to server routes', () => {
|
it('does not attempt client-side navigation to server routes', () => {
|
||||||
return nightmare.goto(`${base}/blog/how-is-sapper-different-from-next`)
|
return nightmare.goto(`${base}/blog/how-is-sapper-different-from-next`)
|
||||||
.init()
|
.init()
|
||||||
|
|||||||
Reference in New Issue
Block a user