Skip to content

Commit

Permalink
Merge pull request #758 from macbre/use-http-server
Browse files Browse the repository at this point in the history
Use http-server for tests (fixes qs security issue)
  • Loading branch information
macbre authored Aug 6, 2020
2 parents a85ecfb + 8289555 commit fe87e29
Show file tree
Hide file tree
Showing 3 changed files with 108 additions and 107 deletions.
208 changes: 105 additions & 103 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 2 additions & 3 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@
"async": "^3.2.0",
"csv-string": "^4.0.1",
"debug": "^4.1.1",
"js-yaml": "^3.13.0",
"js-yaml": "^3.14.0",
"node-statsd": "0.1.1",
"optimist": "^0.6.1",
"optimist-config-file": "^1.0.1",
Expand All @@ -40,9 +40,8 @@
"devDependencies": {
"eslint": "^7.6.0",
"glob": "^7.1.6",
"http-serve": "^1.0.1",
"http-server": "^0.12.3",
"js-beautify": "^1.11.0",
"js-yaml": "^3.14.0",
"mockery": "^2.0.0",
"vows": "^0.8.3"
},
Expand Down
2 changes: 1 addition & 1 deletion test/server-start.sh
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
PORT=8888
DIR=`dirname $0`

$DIR/../node_modules/.bin/http-serve $DIR/webroot -p $PORT -c 84600 --gzip
$DIR/../node_modules/.bin/http-server $DIR/webroot -p $PORT -c 84600 --gzip

0 comments on commit fe87e29

Please sign in to comment.