Skip to content
This repository has been archived by the owner on Jun 27, 2019. It is now read-only.

Commit

Permalink
Change default port 8080 to 80
Browse files Browse the repository at this point in the history
The port 8080 will only be the default test port. Since, the test framework
used does not accept testing our framework in port 80.

Signed-off-by: Bruno Bottazzini <bruno.bottazzini@intel.com>
  • Loading branch information
Bruno Bottazzini committed Mar 18, 2016
1 parent 4061111 commit 8fabd6c
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 3 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ If no argument is provided it will get the default server configuration file.
#### Configuration attributes

server_port:
The port listened by the server. Default: 8080
The port listened by the server. Default: 80

server_output:
Choose if you want the server to show its output.
Expand Down
6 changes: 5 additions & 1 deletion server/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,11 @@

app.use('/', routes);

app.set('port', process.env.PORT || jConf.server_port);
if (server.indexOf("protractor")) {
app.set('port', process.env.PORT || 8080);
} else {
app.set('port', process.env.PORT || jConf.server_port);
}

var server = app.listen(app.get('port'), function() {
if (jConf.server_output) {
Expand Down
2 changes: 1 addition & 1 deletion server/configuration.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"server_port": 8080,
"server_port": 80,
"server_output": true,
"server_user_home_dir": "/tmp",
"sessions_dir" : "./sessions",
Expand Down

0 comments on commit 8fabd6c

Please sign in to comment.