Skip to content

Commit

Permalink
Do not use priviledge port
Browse files Browse the repository at this point in the history
  * make the app on 8000 instead of 80
  * update the Dockerfile and README accordingly
  • Loading branch information
Pamplemousse committed Nov 13, 2016
1 parent 136f73d commit faad235
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@ FROM node:6-onbuild

RUN npm run build

EXPOSE 80
EXPOSE 8000
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,6 @@ npm run build
docker run \
--name xaviermaso.com \
-e NODE_ENV=production \
-p 8080:80 \
-p 8080:8000 \
pamplemousse/xaviermaso.com
```
2 changes: 1 addition & 1 deletion app.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ var bodyParser = require('body-parser')

var app = express()

var server = app.listen(80, function () {
var server = app.listen(8000, function () {
console.log('Listening on port %d', server.address().port)
})

Expand Down

0 comments on commit faad235

Please sign in to comment.