support the case of no TTL when Cache-Control is present

Author: jkyberneees

index.js

@@ -75,7 +75,11 @@ const middleware = (opts) => async (req, res, next) => {
         ttl = cacheControl(payload.headers[CACHE_CONTROL]).maxAge
       }
       if (!ttl) {
-        ttl = Math.max(ms(payload.headers[X_CACHE_TIMEOUT]), 1000) / 1000 // min value: 1 second
+        if (payload.headers[X_CACHE_TIMEOUT]) {
+          ttl = Math.max(ms(payload.headers[X_CACHE_TIMEOUT]), 1000) / 1000 // min value: 1 second
+        } else {
+          return // no TTL found, we don't cache
+        }
       }
 
       // setting cache-control header if absent

test/smoke.test.js

@@ -43,6 +43,12 @@ describe('cache middleware', () => {
       }, 50)
     })
 
+    server.get('/cache-no-maxage', (req, res) => {
+      res.setHeader('etag', '1')
+      res.setHeader('cache-control', 'no-cache')
+      res.send('!maxage')
+    })
+
     server.get('/cache-control', (req, res) => {
       res.setHeader('cache-control', 'max-age=60')
       res.setHeader('etag', '1')
@@ -98,6 +104,13 @@ describe('cache middleware', () => {
     expect(res.headers['x-cache-hit']).to.equal(undefined)
   })
 
+  it('no TTL detected', async () => {
+    await got('http://localhost:3000/cache-no-maxage')
+    const res = await got('http://localhost:3000/cache-no-maxage')
+    expect(res.body).to.equal('!maxage')
+    expect(res.headers['x-cache-hit']).to.equal(undefined)
+  })
+
   it('cache hit (buffer)', async () => {
     const res = await got('http://localhost:3000/cache-buffer')
     expect(res.body).to.equal('world')