feat(response): add content-type header to JSON responses (#80)

Author: jon-cooke

lib/terminus.js

@@ -17,6 +17,7 @@ function noopResolves () {
 
 function sendSuccess (res, info) {
   res.statusCode = 200
+  res.setHeader('Content-Type', 'application/json')
   if (info) {
     return res.end(JSON.stringify({
       status: 'ok',
@@ -28,6 +29,7 @@ function sendSuccess (res, info) {
 
 function sendFailure (res) {
   res.statusCode = 503
+  res.setHeader('Content-Type', 'application/json')
   res.end(FAILURE_RESPONSE)
 }
 

lib/terminus.spec.js

@@ -48,6 +48,8 @@ describe('Terminus', () => {
       fetch('http://localhost:8000/health')
         .then(res => {
           expect(res.status).to.eql(200)
+          expect(res.headers.has('Content-Type')).to.eql(true)
+          expect(res.headers.get('Content-Type')).to.eql('application/json')
           expect(onHealthCheckRan).to.eql(true)
           done()
         })
@@ -72,6 +74,8 @@ describe('Terminus', () => {
       fetch('http://localhost:8000/health')
         .then(res => {
           expect(res.status).to.eql(200)
+          expect(res.headers.has('Content-Type')).to.eql(true)
+          expect(res.headers.get('Content-Type')).to.eql('application/json')
           expect(onHealthCheckRan).to.eql(true)
           return res.json()
         })