-
-
Notifications
You must be signed in to change notification settings - Fork 216
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bug] A matcher with multiple eval() doesn't work #315
Comments
JoshuaCWebDeveloper
added a commit
to JoshuaCWebDeveloper/node-casbin
that referenced
this issue
Aug 16, 2021
…l() w/ matching ruleName This fixes casbin#315. fix casbin#315
working on this |
🎉 This issue has been resolved in version 5.11.4 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
A matcher with multiple eval() statements only calls the first eval(), caching the results.
This is likely the same issue as casbin/casbin#619, which was fixed in that repo, but apparently not in this one. There are also several people experiencing this issue here: http://disq.us/p/2g9ubnp (although they didn't say which language).
I'm experiencing this issue in node-casbin; however, I can also reproduce it in the online editor: https://casbin.org/casbin-editor/#6FGL6FCSV (not sure which language it uses). In this example, I would expect the result to be false:
eval()
should evaluate10 > 18
, however, it instead evaluates the first result again (the array) which is truthy.The text was updated successfully, but these errors were encountered: