-
Notifications
You must be signed in to change notification settings - Fork 15
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
feat: add casbin engine support #31
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: closetool <[email protected]>
@hsluoyz @chrisohaver plz review |
Sorry - this had escaped my radar until now. Will review when I get a chance. I will say though, that I have been considering breaking all policy engine plugins out into separate repositories, which would work better with Go modules. |
Hi @chrisohaver any update? |
Thanks for the ping. I have decided to reorganize the firewall plugin code, and split the policy engine plugins out into separate repositories. I'll try to get this done soon. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This repository includes two Policy Engine Plugins:
should be:
This repository includes three Policy Engine Plugins:
Signed-off-by: closetool [email protected]
Add casbin policy engine support
Fix: casbin/casbin#828