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

fix: replace fnmatch with wcmatch for glob match, keep same behavior with Go Casbin #149

Merged
merged 3 commits into from
May 12, 2021

Conversation

Zxilly
Copy link
Contributor

@Zxilly Zxilly commented May 11, 2021

fix: #148

Signed-off-by: Zxilly zhouxinyu1001@gmail.com

ref to discussion at https://bugs.python.org/issue28718, it'll be complicated to implement it.

So I have to introduce a new dependency wcmatch.

Signed-off-by: Zxilly <zhouxinyu1001@gmail.com>
Signed-off-by: Zxilly <zhouxinyu1001@gmail.com>
@Zxilly
Copy link
Contributor Author

Zxilly commented May 11, 2021

@hsluoyz @techoner plz review

@hsluoyz
Copy link
Member

hsluoyz commented May 11, 2021

@Zxilly plz fix:

image

I saw you already sent and merged a PR to remove Py 3.5. Maybe you need to open a new PR to "refresh" the checks.

Signed-off-by: Zxilly <zhouxinyu1001@gmail.com>
@hsluoyz hsluoyz requested a review from leeqvip May 12, 2021 10:55
requirements.txt Show resolved Hide resolved
casbin/util/builtin_operators.py Show resolved Hide resolved
@Zxilly Zxilly requested a review from hsluoyz May 12, 2021 11:04
@hsluoyz hsluoyz changed the title fix: keep same behavior with go-casbin fix: replace fnmatch with wcmatch for glob match, keep same behavior with Go Casbin May 12, 2021
@hsluoyz hsluoyz merged commit 906f40c into casbin:master May 12, 2021
github-actions bot pushed a commit that referenced this pull request May 12, 2021
## [1.0.5](v1.0.4...v1.0.5) (2021-05-12)

### Bug Fixes

* replace fnmatch with wcmatch for glob match, keep same behavior with Go Casbin ([#149](#149)) ([906f40c](906f40c))
@hsluoyz
Copy link
Member

hsluoyz commented May 12, 2021

🎉 This PR is included in version 1.0.5 🎉

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
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

glob_match() is different from other languages
2 participants