casbin 5.27.1 → 5.29.0

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+# [5.29.0](https://github.com/casbin/node-casbin/compare/v5.28.0...v5.29.0) (2024-04-10)
+
+
+### Features
+
+* update CI Node.js version to 18 and 20 ([4aa74ab](https://github.com/casbin/node-casbin/commit/4aa74ab710aa300c172c71b802453dd64b3ec1a7))
+
+# [5.28.0](https://github.com/casbin/node-casbin/compare/v5.27.1...v5.28.0) (2023-11-21)
+
+
+### Features
+
+* fix WatcherEx not triggered issue ([61c1595](https://github.com/casbin/node-casbin/commit/61c159533904f38d82eb0418a1de949666cc381a))
+
 ## [5.27.1](https://github.com/casbin/node-casbin/compare/v5.27.0...v5.27.1) (2023-09-19)
 
 

package/README.md

@@ -14,6 +14,20 @@
 [download-image]: https://img.shields.io/npm/dm/casbin.svg?style=flat-square
 [download-url]: https://npmjs.org/package/casbin
 
+<p align="center">
+  <sup>Sponsored by</sup>
+  <br>
+  <a href="https://stytch.com/docs?utm_source=oss-sponsorship&utm_medium=paid_sponsorship&utm_campaign=casbin">
+    <picture>
+      <source media="(prefers-color-scheme: dark)" srcset="https://cdn.casbin.org/img/stytch-white.png">
+      <source media="(prefers-color-scheme: light)" srcset="https://cdn.casbin.org/img/stytch-charcoal.png">
+      <img src="https://cdn.casbin.org/img/stytch-charcoal.png" width="275">
+    </picture>
+  </a><br/>
+  <a href="https://stytch.com/docs?utm_source=oss-sponsorship&utm_medium=paid_sponsorship&utm_campaign=casbin"><b>Build auth with fraud prevention, faster.</b><br/> Try Stytch for API-first authentication, user & org management, multi-tenant SSO, MFA, device fingerprinting, and more.</a>
+  <br>
+</p>
+
 💖 [**Looking for an open-source identity and access management solution like Okta, Auth0, Keycloak ? Learn more about: Casdoor**](https://casdoor.org/)
 
 <a href="https://casdoor.org/"><img src="https://user-images.githubusercontent.com/3787410/147868267-6ac74908-5654-4f9c-ac79-8852af9ff925.png" alt="casdoor" style="width: 50%; height: 50%"/></a>

package/lib/cjs/internalEnforcer.js

@@ -117,10 +117,12 @@ class InternalEnforcer extends coreEnforcer_1.CoreEnforcer {
             }
         }
         if (useWatcher) {
-            if (this.watcher && this.autoNotifyWatcher) {
+            if (this.autoNotifyWatcher) {
                 // In fact I think it should wait for the respond, but they implement add_policy() like this
                 // error intentionally ignored
-                this.watcher.update();
+                if (this.watcher) {
+                    this.watcher.update();
+                }
             }
         }
         const ok = this.model.updatePolicy(sec, ptype, oldRule, newRule);
@@ -148,7 +150,7 @@ class InternalEnforcer extends coreEnforcer_1.CoreEnforcer {
             }
         }
         if (useWatcher) {
-            if (this.watcher && this.autoNotifyWatcher) {
+            if (this.autoNotifyWatcher) {
                 // error intentionally ignored
                 if (this.watcherEx) {
                     this.watcherEx.updateForRemovePolicy(sec, ptype, ...rule);
@@ -187,7 +189,7 @@ class InternalEnforcer extends coreEnforcer_1.CoreEnforcer {
             }
         }
         if (useWatcher) {
-            if (this.watcher && this.autoNotifyWatcher) {
+            if (this.autoNotifyWatcher) {
                 // error intentionally ignored
                 if (this.watcherEx) {
                     this.watcherEx.updateForRemovePolicies(sec, ptype, ...rules);
@@ -218,7 +220,7 @@ class InternalEnforcer extends coreEnforcer_1.CoreEnforcer {
             }
         }
         if (useWatcher) {
-            if (this.watcher && this.autoNotifyWatcher) {
+            if (this.autoNotifyWatcher) {
                 // error intentionally ignored
                 if (this.watcherEx) {
                     this.watcherEx.updateForRemoveFilteredPolicy(sec, ptype, fieldIndex, ...fieldValues);

package/lib/esm/internalEnforcer.js

@@ -114,10 +114,12 @@ export class InternalEnforcer extends CoreEnforcer {
             }
         }
         if (useWatcher) {
-            if (this.watcher && this.autoNotifyWatcher) {
+            if (this.autoNotifyWatcher) {
                 // In fact I think it should wait for the respond, but they implement add_policy() like this
                 // error intentionally ignored
-                this.watcher.update();
+                if (this.watcher) {
+                    this.watcher.update();
+                }
             }
         }
         const ok = this.model.updatePolicy(sec, ptype, oldRule, newRule);
@@ -145,7 +147,7 @@ export class InternalEnforcer extends CoreEnforcer {
             }
         }
         if (useWatcher) {
-            if (this.watcher && this.autoNotifyWatcher) {
+            if (this.autoNotifyWatcher) {
                 // error intentionally ignored
                 if (this.watcherEx) {
                     this.watcherEx.updateForRemovePolicy(sec, ptype, ...rule);
@@ -184,7 +186,7 @@ export class InternalEnforcer extends CoreEnforcer {
             }
         }
         if (useWatcher) {
-            if (this.watcher && this.autoNotifyWatcher) {
+            if (this.autoNotifyWatcher) {
                 // error intentionally ignored
                 if (this.watcherEx) {
                     this.watcherEx.updateForRemovePolicies(sec, ptype, ...rules);
@@ -215,7 +217,7 @@ export class InternalEnforcer extends CoreEnforcer {
             }
         }
         if (useWatcher) {
-            if (this.watcher && this.autoNotifyWatcher) {
+            if (this.autoNotifyWatcher) {
                 // error intentionally ignored
                 if (this.watcherEx) {
                     this.watcherEx.updateForRemoveFilteredPolicy(sec, ptype, fieldIndex, ...fieldValues);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "casbin",
-  "version": "5.27.1",
+  "version": "5.29.0",
   "description": "An authorization library that supports access control models like ACL, RBAC, ABAC in Node.JS",
   "main": "lib/cjs/index.js",
   "typings": "lib/cjs/index.d.ts",