Skip to content

Commit

Permalink
fix: use the main prop for the application entry point.
Browse files Browse the repository at this point in the history
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 https://github.com/bucharest-gold/centos7-s2i-nodejs/issues/33\#issuecomment-382587104.

fixes #98
  • Loading branch information
lholmquist committed Apr 23, 2018
1 parent 3550fa4 commit 34b8c91
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion bin/www
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ const http = require('http');
* Get port from environment and store in Express.
*/

const port = normalizePort(process.env.PORT || '3000');
const port = normalizePort(process.env.PORT || '8080');
app.set('port', port);

/**
Expand Down
3 changes: 2 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,9 @@
"release": "standard-version -a",
"openshift": "nodeshift --strictSSL=false --nodeVersion=8.x",
"postinstall": "license-reporter report -s && license-reporter save -s --xml licenses.xml",
"start": "PORT=8080 node ./bin/www"
"start": "node ."
},
"main": "./bin/www",
"standard-version": {
"scripts": {
"postbump": "npm run postinstall && node release.js",
Expand Down

0 comments on commit 34b8c91

Please sign in to comment.