Expose filenaming mechanism

Author: notVitaliy

.gitignore

@@ -3,5 +3,8 @@
 /test/_fixtures/https%3A__openstax.org_*_GET_*_body.raw
 /test/_fixtures/https%3A__openstax.org_*_GET_*_options.json
 
+/test/_fixtures/https%3A__jsonplaceholder.typicode.com_users_*.raw
+/test/_fixtures/https%3A__jsonplaceholder.typicode.com_users_*.json
+
 /browser-bundle.js
 /browser-bundle.js.map

README.md

@@ -84,6 +84,19 @@ fetch('https://weedmaps.com/sitemap') // <-- This will be ignored from vcr
 })
 ```
 
+## How do I define custom fixture filenames?
+
+There are 3 find/replace variables exposed `[url]`, `[method]`, and `[hash]`.
+
+```js
+// import fetch from 'fetch';
+import fetch from 'fetch-vcr';
+
+fetch.configure({
+  fixtureName: '[url]_[method]_[hash]', // <-- This is the default value
+})
+```
+
 ## Jest Setup
 
 Just add the following to `package.json`:

lib/index.js

@@ -21,7 +21,8 @@ var CONFIGURATION = {
   mode: VCR_MODE,
   fixturePath: './_fixtures',
   headerBlacklist: ['authorization', 'user-agent'], // These need to be lowercase
-  ignoreUrls: [] // regex of urls to ignore
+  ignoreUrls: [], // regex of urls to ignore
+  fixtureName: '[url]_[method]_[hash]'
 }
 
 function debug(url, message) {
@@ -91,13 +92,19 @@ function buildHash(url, args) {
 }
 
 function buildFilenamePrefix(url, args, hash) {
-  args = args || { }
   var [baseUrl, query] = url.split('?')
   url = escape(baseUrl).replace(/\//g, '_')
-  var method = args.method || 'GET'
+  url += query ? '_' + hashCode(query) : ''
+
+  var method = (args || { }).method || 'GET'
   method = method.toUpperCase()
-  var paramsHash = query ? '_' + hashCode(query) : ''
-  return url + paramsHash + '_' + method + '_' + hash
+
+  var fixtureName = CONFIGURATION.fixtureName
+
+  return fixtureName
+    .replace(/\[url\]/g, url)
+    .replace(/\[method\]/g, method)
+    .replace(/\[hash\]/g, hash)
 }
 
 function buildOptionsFilename(url, args, hash) {
@@ -244,6 +251,7 @@ fetchVCR.configure = function (config) {
   CONFIGURATION.mode = VCR_MODE || config.mode
   CONFIGURATION.fixturePath = config.fixturePath || CONFIGURATION.fixturePath
   CONFIGURATION.ignoreUrls = config.ignoreUrls || CONFIGURATION.ignoreUrls
+  CONFIGURATION.fixtureName = config.fixtureName || CONFIGURATION.fixtureName
   if (config.headerBlacklist) {
     CONFIGURATION.headerBlacklist = []
     config.headerBlacklist.forEach(function (key) {