Browse Source

calculateResponseMaxAge only needs headers as arg

pull/14/head
Luke Childs 8 years ago
parent
commit
edef446641
  1. 10
      src/index.js

10
src/index.js

@ -35,16 +35,16 @@ class Onionoo {
} }
// Returns cache max age from response headers // Returns cache max age from response headers
calculateResponseMaxAge (response) { calculateResponseMaxAge (headers) {
// Get max age ms // Get max age ms
const cacheControl = response.headers['cache-control'] const cacheControl = headers['cache-control']
const maxAgeRegex = /max-age=(\d+)/ const maxAgeRegex = /max-age=(\d+)/
let maxAge = cacheControl && cacheControl.match(maxAgeRegex) let maxAge = cacheControl && cacheControl.match(maxAgeRegex)
maxAge = maxAge ? maxAge[1] : 0 maxAge = maxAge ? maxAge[1] : 0
// Take current age into account // Take current age into account
if (response.headers.age) { if (headers.age) {
maxAge -= response.headers.age maxAge -= headers.age
} }
// Don't return negative values // Don't return negative values
@ -89,7 +89,7 @@ class Onionoo {
// Cache response // Cache response
if (this.options.cache) { if (this.options.cache) {
const ttl = this.calculateResponseMaxAge(response) const ttl = this.calculateResponseMaxAge(response.headers)
if (ttl) { if (ttl) {
this.options.cache.set(url, response, { ttl }) this.options.cache.set(url, response, { ttl })
} }

Loading…
Cancel
Save