feathers-ucan 0.1.42 → 0.1.44

package/lib/env/version.d.ts

@@ -1 +1 @@
-export declare const VERSION = "0.1.42";
+export declare const VERSION = "0.1.44";

package/lib/index.cjs

@@ -955,12 +955,14 @@ const checkUcan = (requiredCapabilities, options) => {
         } = options || {
           loginPass: [[['*'], ['nonExistentMethod']]]
         };
+        if (options != null && options.log) console.log('loginPass precheck', loginPass);
         if (loginPass != null && loginPass.length) {
           //object of scrubbed data object for pass that includes only limited access or full context.data object if no limits were present
           let scrubbedData = {};
           //scruData defaults to true - is only set to false
           let scrubData = true;
           const checkLoginPass = async lpass => {
+            if (options != null && options.log) console.log('checkLoginPass', lpass);
             let methodsOnly = [];
             const allMethods = lpass[1] === '*';
             let methodIdx = -1;
@@ -979,7 +981,7 @@ const checkUcan = (requiredCapabilities, options) => {
               /** function for comparing record login id with context login*/
               const checkLogin = (recordLoginPassId, loginIdPath = '_id') => {
                 const loginCheckId = symbolUcan._get(context.params, `${configuration.entity}.${loginIdPath}`);
-                if (options != null && options.log) console.log('checkLoginPass', loginCheckId, recordLoginPassId);
+                if (options != null && options.log) console.log('checkLogin', loginCheckId, recordLoginPassId);
                 /**Make sure both are present to avoid pass on undefined*/
                 if (loginCheckId && recordLoginPassId) {
                   /** change login path result to array no matter what */
@@ -990,7 +992,11 @@ const checkUcan = (requiredCapabilities, options) => {
                       const checkId = String(checkArr[i]);
                       for (let rl = 0; rl < recordLoginPassId.length;) {
                         const rlId = String(recordLoginPassId[rl]);
-                        if (rlId === checkId) loginOk = true;else rl++;
+                        if (rlId === checkId) {
+                          // Found a matching login id; mark success and break out of the inner loop
+                          loginOk = true;
+                          break;
+                        } else rl++;
                       }
                       if (loginOk) return;
                     }
@@ -1003,6 +1009,7 @@ const checkUcan = (requiredCapabilities, options) => {
                 context = setExists(context, existing);
                 /**perform the check*/
                 const idArr = Array.isArray(lpass[2]) ? lpass[2] : lpass[0] || [];
+                if (options != null && options.log) console.log('loginPass: id paths to check', idArr);
                 for (const passPath of idArr || []) {
                   const spl = String(passPath).split('/');
                   if (spl[0].includes('*')) {
@@ -1025,6 +1032,11 @@ const checkUcan = (requiredCapabilities, options) => {
                     }
                   } else checkLogin(symbolUcan._get(existing, spl[0]), spl[1] || '_id');
                 }
+              } else {
+                if (options != null && options.log) console.log('loginPass: no existing record loaded; skipping checkLogin', {
+                  method: context.method,
+                  path: context.path
+                });
               }
               if (loginOk) {
                 v.ok = true;