@bedrock/kms 10.1.0 → 10.2.0

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # bedrock-kms ChangeLog
 
+## 10.2.0 - 2022-05-13
+
+### Added
+- Expose `_disableClearCacheOnUpdate` for testing cache busting only; do not use in
+  production.
+
 ## 10.1.0 - 2022-05-13
 
 ### Added

package/lib/keystores.js

@@ -16,6 +16,7 @@ import './config.js';
 
 const USAGE_COUNTER_MAX_CONCURRENCY = 100;
 let KEYSTORE_CONFIG_CACHE;
+let DISABLE_CLEAR_CACHE_ON_UPDATE = false;
 
 // cache only exported for testing purposes
 export {KEYSTORE_CONFIG_CACHE as _KEYSTORE_CONFIG_CACHE};
@@ -198,8 +199,10 @@ export async function update({config, explain = false} = {}) {
       });
   }
 
-  // delete record from cache
-  KEYSTORE_CONFIG_CACHE.delete(config.id);
+  if(!DISABLE_CLEAR_CACHE_ON_UPDATE) {
+    // delete record from cache
+    KEYSTORE_CONFIG_CACHE.delete(config.id);
+  }
 
   return true;
 }
@@ -210,7 +213,7 @@ export async function update({config, explain = false} = {}) {
  * @param {object} options - The options to use.
  * @param {string} options.id - The ID of the keystore.
  * @param {boolean} [options.fresh=false] - False if it is safe to use a
- *   potentially cached value, false to always get a fresh value.
+ *   potentially cached value, true to always get a fresh value.
  *
  * @returns {Promise<object>} Resolves to `{config, meta}`.
  */
@@ -335,3 +338,8 @@ export async function _getUncachedRecord({id, explain = false} = {}) {
   }
   return record;
 }
+
+// exported for testing purposes
+export function _disableClearCacheOnUpdate(disable) {
+  DISABLE_CLEAR_CACHE_ON_UPDATE = disable;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bedrock/kms",
-  "version": "10.1.0",
+  "version": "10.2.0",
   "type": "module",
   "description": "Key management for Bedrock applications",
   "main": "./lib/index.js",