casbin / casbin-mesh

A scalable authorization layer built on Casbin (The experimental project is currently unstable)
https://mesh.casbin.org
Apache License 2.0
25 stars 13 forks source link

fix: RemoveFilteredPolicy raise service exception when fieldIndex=0 #70

Closed zhouzhuangjie closed 1 year ago

casbin-bot commented 1 year ago

@WenyXu @nodece please review

CLAassistant commented 1 year ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 2 committers have signed the CLA.

:x: justin
:x: ericchenCW


justin seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

hsluoyz commented 1 year ago

@zhouzhuangjie fix:

image

hsluoyz commented 1 year ago

@zhouzhuangjie what does this PR do? Why change int to int pointer?