fix(auth): condition if (!params.cache) was treating undefined as "cache disabled", when it should only disable cache when explicitly set to false.

Ansonhkg · Ansonhkg · commit bd8240e72285 · 2025-07-29T18:02:36.000+01:00
diff --git a/packages/auth/src/lib/AuthManager/try-getters/tryGetCachedDelegationAuthSig.ts b/packages/auth/src/lib/AuthManager/try-getters/tryGetCachedDelegationAuthSig.ts
@@ -13,7 +13,7 @@ export async function tryGetCachedDelegationAuthSig(params: {
   cache?: boolean; // If false, always generate a new auth sig
 }) {
   // If cache is explicitly disabled, skip cache reading and generate new auth sig
-  if (!params.cache) {
+  if (params.cache === false) {
     _logger.info(
       'tryGetCachedDelegationAuthSig: Cache disabled, generating new delegation auth sig.',
       {

Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@ export async function tryGetCachedDelegationAuthSig(params: {`
`13`	`13`	`cache?: boolean; // If false, always generate a new auth sig`
`14`	`14`	`}) {`
`15`	`15`	`// If cache is explicitly disabled, skip cache reading and generate new auth sig`
`16`		`- if (!params.cache) {`
	`16`	`+ if (params.cache === false) {`
`17`	`17`	`_logger.info(`
`18`	`18`	`'tryGetCachedDelegationAuthSig: Cache disabled, generating new delegation auth sig.',`
`19`	`19`	`{`