fix(config): allow to override cache key per request (lisaogren#237)

Author: lisathesecond

src/config.js

@@ -105,8 +105,13 @@ const mergeRequestConfig = function (config, req) {
  mergedConfig.debug = debug
  }
 
+ // Create a cache key method
+ if (requestConfig.key) {
+ mergedConfig.key = key(requestConfig)
+ }
+
  // Generate request UUID
- mergedConfig.uuid = config.key(req)
+ mergedConfig.uuid = mergedConfig.key(req)
 
  config.debug(`Request config for ${req.url}`, mergedConfig)
 

test/spec/config.spec.js

@@ -1,6 +1,7 @@
 /* globals describe it */
 
 import assert from 'assert'
+import isFunction from 'lodash/isFunction'
 
 import config from 'src/config'
 
@@ -54,7 +55,8 @@ describe('Per request config', () => {
  const mergedConfig = config.mergeRequestConfig(globalConfig, fakeRequest)
 
  assert.strictEqual(mergedConfig.maxAge, requestConfig.maxAge)
- assert.strictEqual(mergedConfig.key, requestConfig.key)
+ assert.ok(isFunction(mergedConfig.key))
+ assert.strictEqual(mergedConfig.key({ url: 'url' }), 'myKey/url')
  assert.deepStrictEqual(mergedConfig.exclude, {
  ...globalConfig.exclude,
  ...requestConfig.exclude