terribleplan/next.js
1
0
Fork 0
mirror of https://github.com/terribleplan/next.js.git synced 2024-01-19 02:48:18 +00:00
Code Issues Releases Wiki Activity

Fix typos on next-server.js comments. (#5672)

Browse source
This commit is contained in:
Anderson Leite 2018-11-14 00:55:25 -08:00 committed by Tim Neutkens
parent e930d1e821
commit 0d77dda28c
1 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
packages/next-server/server/next-server.js
View file

@ -120,9 +120,9 @@ export default class Server {
} }
}, },
{ {
// It's very important keep this route's param optional. // It's very important to keep this route's param optional.
// (but it should support as many as params, seperated by '/') // (but it should support as many params as needed, separated by '/')
// Othewise this will lead to a pretty simple DOS attack. // Otherwise this will lead to a pretty simple DOS attack.
// See more: https://github.com/zeit/next.js/issues/2617 // See more: https://github.com/zeit/next.js/issues/2617
match: route('/static/:path*'), match: route('/static/:path*'),
fn: async (req, res, params) => { fn: async (req, res, params) => {
@ -133,9 +133,9 @@ export default class Server {
] ]
if (this.nextConfig.useFileSystemPublicRoutes) { if (this.nextConfig.useFileSystemPublicRoutes) {
// It's very important keep this route's param optional. // It's very important to keep this route's param optional.
// (but it should support as many as params, seperated by '/') // (but it should support as many params as needed, separated by '/')
// Othewise this will lead to a pretty simple DOS attack. // Otherwise this will lead to a pretty simple DOS attack.
// See more: https://github.com/zeit/next.js/issues/2617 // See more: https://github.com/zeit/next.js/issues/2617
routes.push({ routes.push({
match: route('/:path*'), match: route('/:path*'),