search

casbin / Casbin.NET

An authorization library that supports access control models like ACL, RBAC, ABAC in .NET (C#)
https://casbin.org
Apache License 2.0
1.13k stars 110 forks source link

Lacked exist check when add policy #348

Closed sagilio0728 closed 3 months ago

sagilio0728 commented 3 months ago

The HasPolicy only exists at InternalAddPolicyAync, but lacks at InternalAddPolicy: https://github.com/casbin/Casbin.NET/blob/33063757b27dfdf1256bbed7e0ea93ced99ac197/Casbin/Extensions/Enforcer/InternalEnforcerExtension.cs#L46-L87

It will cause duplicate policy when adding policy. From: https://github.com/casbin-net/efcore-adapter/issues/69

casbin-bot commented 3 months ago

@sagilio @sociometry @AsakusaRinne

github-actions[bot] commented 3 months ago

:tada: This issue has been resolved in version 2.5.2 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: