643447ed03
This addresses a bug where invoking `next dev` or `next start` with `--port 0` would fall back to the default port of 3000 instead of binding to port 0 (which typically results in the operating system assigning a free port). I couldn't find a straightforward way of adding a test for next-start. It looks like we could add a similar test as for dev, but would need to generate a built project to serve. Manual test plan for `next start`: ``` $ ./packages/next/dist/bin/next start --port 0 ready - started server on 0.0.0.0:53508, url: http://localhost:53508 ``` ## Bug - [ ] Related issues linked using `fixes #number` - [x] Integration tests added - [ ] Errors have helpful link attached, see `contributing.md` ## Documentation / Examples - [x] Make sure the linting passes by running `pnpm lint` - [ ] The examples guidelines are followed from [our contributing doc](https://github.com/vercel/next.js/blob/canary/contributing.md#adding-examples) Co-authored-by: Steven <steven@ceriously.com> Co-authored-by: JJ Kasper <jj@jjsweb.site> Co-authored-by: Wyatt Johnson <accounts+github@wyattjoh.ca>
92 lines
2.5 KiB
JavaScript
Executable file
92 lines
2.5 KiB
JavaScript
Executable file
#!/usr/bin/env node
|
|
|
|
import arg from 'next/dist/compiled/arg/index.js'
|
|
import { startServer } from '../server/lib/start-server'
|
|
import { getPort, printAndExit } from '../server/lib/utils'
|
|
import * as Log from '../build/output/log'
|
|
import isError from '../lib/is-error'
|
|
import { getProjectDir } from '../lib/get-project-dir'
|
|
import { cliCommand } from '../lib/commands'
|
|
|
|
const nextStart: cliCommand = (argv) => {
|
|
const validArgs: arg.Spec = {
|
|
// Types
|
|
'--help': Boolean,
|
|
'--port': Number,
|
|
'--hostname': String,
|
|
'--keepAliveTimeout': Number,
|
|
|
|
// Aliases
|
|
'-h': '--help',
|
|
'-p': '--port',
|
|
'-H': '--hostname',
|
|
}
|
|
let args: arg.Result<arg.Spec>
|
|
try {
|
|
args = arg(validArgs, { argv })
|
|
} catch (error) {
|
|
if (isError(error) && error.code === 'ARG_UNKNOWN_OPTION') {
|
|
return printAndExit(error.message, 1)
|
|
}
|
|
throw error
|
|
}
|
|
if (args['--help']) {
|
|
console.log(`
|
|
Description
|
|
Starts the application in production mode.
|
|
The application should be compiled with \`next build\` first.
|
|
|
|
Usage
|
|
$ next start <dir> -p <port>
|
|
|
|
<dir> represents the directory of the Next.js application.
|
|
If no directory is provided, the current directory will be used.
|
|
|
|
Options
|
|
--port, -p A port number on which to start the application
|
|
--hostname, -H Hostname on which to start the application (default: 0.0.0.0)
|
|
--keepAliveTimeout Max milliseconds to wait before closing inactive connections
|
|
--help, -h Displays this message
|
|
`)
|
|
process.exit(0)
|
|
}
|
|
|
|
const dir = getProjectDir(args._[0])
|
|
const host = args['--hostname'] || '0.0.0.0'
|
|
const port = getPort(args)
|
|
|
|
const keepAliveTimeoutArg: number | undefined = args['--keepAliveTimeout']
|
|
if (
|
|
typeof keepAliveTimeoutArg !== 'undefined' &&
|
|
(Number.isNaN(keepAliveTimeoutArg) ||
|
|
!Number.isFinite(keepAliveTimeoutArg) ||
|
|
keepAliveTimeoutArg < 0)
|
|
) {
|
|
printAndExit(
|
|
`Invalid --keepAliveTimeout, expected a non negative number but received "${keepAliveTimeoutArg}"`,
|
|
1
|
|
)
|
|
}
|
|
|
|
const keepAliveTimeout = keepAliveTimeoutArg
|
|
? Math.ceil(keepAliveTimeoutArg)
|
|
: undefined
|
|
|
|
startServer({
|
|
dir,
|
|
hostname: host,
|
|
port,
|
|
keepAliveTimeout,
|
|
})
|
|
.then(async (app) => {
|
|
const appUrl = `http://${app.hostname}:${app.port}`
|
|
Log.ready(`started server on ${host}:${app.port}, url: ${appUrl}`)
|
|
await app.prepare()
|
|
})
|
|
.catch((err) => {
|
|
console.error(err)
|
|
process.exit(1)
|
|
})
|
|
}
|
|
|
|
export { nextStart }
|