Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add Casbin-RS to our feature set table #71

Closed
hsluoyz opened this issue Mar 17, 2020 · 7 comments · Fixed by casbin/casbin-website#60
Closed

Add Casbin-RS to our feature set table #71

hsluoyz opened this issue Mar 17, 2020 · 7 comments · Fixed by casbin/casbin-website#60
Labels
documentation Improvements or additions to documentation help wanted Extra attention is needed

Comments

@hsluoyz
Copy link
Member

hsluoyz commented Mar 17, 2020

See: https://casbin.org/docs/en/overview#feature-set-for-different-languages

image
@hsluoyz hsluoyz added documentation Improvements or additions to documentation help wanted Extra attention is needed labels Mar 17, 2020
@hsluoyz
Copy link
Member Author

hsluoyz commented Mar 18, 2020

@GopherJ @xcaptain @PsiACE

@GopherJ
Copy link
Member

GopherJ commented Apr 5, 2020

@hsluoyz I would say we have all except ABAC and FilteredAdapter, about ABAC @xcaptain may know more than me, I have never used it so I have no idea.

@xcaptain
Copy link
Contributor

xcaptain commented Apr 5, 2020

@GopherJ casbin-rs doesn't support ABAC yet, because we are using Vec<&str> as enforce parameter, which requires all elements in the vector have the same type, while ABAC requires matching on some attributes of an object.

Do you want to make a pull request to the doc site about the current status of casbin-rs?

@hsluoyz
Copy link
Member Author

hsluoyz commented Apr 5, 2020

Thanks @GopherJ @xcaptain ! Can anyone update the docs?

@xcaptain
Copy link
Contributor

xcaptain commented Apr 5, 2020

I think the FilteredAdapter would be much easier to implement, do we need to implement it first?

@xcaptain
Copy link
Contributor

xcaptain commented Apr 5, 2020

@hsluoyz casbin/casbin-website#60 please review

@xcaptain xcaptain mentioned this issue Apr 5, 2020
Merged
@hsluoyz
Copy link
Member Author

hsluoyz commented Apr 5, 2020

Merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
documentation Improvements or additions to documentation help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

update casbin-rs status xcaptain/casbin-website
3 participants
@hsluoyz @xcaptain @GopherJ