Skip to content

Commit 0911140

Browse files
committed
fix: use the main prop for the application entry point.
npm start will now look at the main property for the entry point. Port 8080 is now set as the default port instead of 3000. This is a fix related to nodeshift-archived/centos7-s2i-nodejs#33\#issuecomment-382587104. fixes nodeshift-archived#93
1 parent 1289c4b commit 0911140

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

bin/www

+1-1
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ const http = require('http');
3232
* Get port from environment and store in Express.
3333
*/
3434

35-
const port = normalizePort(process.env.PORT || '3000');
35+
const port = normalizePort(process.env.PORT || '8080');
3636
app.set('port', port);
3737

3838
/**

package.json

+2-1
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,9 @@
1818
"release": "standard-version -a",
1919
"openshift": "nodeshift --strictSSL=false --dockerImage=registry.access.redhat.com/rhoar-nodejs/nodejs-8",
2020
"postinstall": "license-reporter report -s && license-reporter save -s --xml licenses.xml",
21-
"start": "PORT=8080 node ./bin/www"
21+
"start": "node ."
2222
},
23+
"main": "./bin/www",
2324
"standard-version": {
2425
"scripts": {
2526
"postbump": "npm run postinstall && node release.js",

0 commit comments

Comments
 (0)