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.

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: support updating policy #47

Merged
merged 1 commit into from
Feb 11, 2022
Merged

feat: support updating policy #47

merged 1 commit into from
Feb 11, 2022

Conversation

Abingcbc
Copy link
Member

@Abingcbc Abingcbc commented Feb 11, 2022

Signed-off-by: abingcbc [email protected]

Fix: #45

Notes: MongoDB starts to support transactions from v4. But txn is only allowed with a replica set.
So in mongodb-adapter, for standalone MongoDB server, UpdateFilteredPolicies is working in a non-transactional way which is unsafe and not recommended.

@casbin-bot
Copy link
Member

@tangyang9464 @closetool @sagilio please review

@coveralls
Copy link

Coverage Status

Coverage decreased (-5.1%) to 83.37% when pulling 76772af on Abingcbc:update into b207188 on casbin:master.

@hsluoyz hsluoyz merged commit a1f441b into casbin:master Feb 11, 2022
@github-actions
Copy link

🎉 This PR is included in version 3.3.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
4 participants