-
Notifications
You must be signed in to change notification settings - Fork 23
Conversation
affects: tessellate-server
affects: tessellate-server
affects: tessellate-server
affects: tessellate-server
affects: tessellate-server
…files. affects: tessellate-server Part of #39.
affects: tessellate-server
affects: tessellate-server
affects: tessellate-server
Changes Unknown when pulling e99fb6e on 37-improve-server into ** on master**. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Could you provide a README on how to use tessellate-server and what is its purpose? Or do you prefer a new ticket for that?
|
||
this.app | ||
.use(morgan(morganFormat, {skip: morganSkip})) | ||
.use(error) | ||
.use(error()) | ||
.use(bodyParser({enableTypes: ['json'], strict: true})) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
strict
already has a default value of true
, we should keep configuration at minimum.
|
||
const [appServer, metricsServer] = await Promise.all([ | ||
startServer(this.app.callback(), port), | ||
startServer(this.metrics.callback(), metricsPort) | ||
startServer(this.app.callback(), parseInt(port)), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you should extract parseInt(port)
as a constant
affects: tessellate-server
9a2f5fc
to
21c7948
Compare
Changes Unknown when pulling 21c7948 on 37-improve-server into ** on master**. |
Changes Unknown when pulling 21c7948 on 37-improve-server into ** on master**. |
Improve tessellate-server in preparation for #37.