search

casbin-net / casbin-aspnetcore

Casbin.NET integration middleware and sample code for ASP.NET Core
https://github.com/casbin/Casbin.NET
Apache License 2.0
64 stars 20 forks source link

fix: Cannot resolve symbol 'SyncedEnforcer' #57

Closed ZhengLin-Li closed 1 year ago

ZhengLin-Li commented 1 year ago

fix: https://github.com/casbin-net/casbin-aspnetcore/issues/55

fix: Cannot resolve symbol 'SyncedEnforcer'

Actually, there is no SyncedEnforcer class in this project. I have searched the keyword SyncedEnforcer and let my IDE find them in files, and it turned out the keyword SyncedEnforcer only appears three times.

So, I believe this issue may be caused by some copy-paste problems. I will fix it in this PR.

casbin-bot commented 1 year ago

@sagilio @AsakusaRinne please review

sagilio commented 1 year ago

@ZhengLin-Li These changes look good, and we may aslo need to upgrade the Casbin.NET to the latest valid version (2.0.0-preview.5). SyncedEnforcer is the thread-safe version defined in the old preview version of Casbin.NET package. We delete it because the default enforcer is already thread-safe at the new version.

sagilio commented 1 year ago

This commit https://github.com/casbin-net/casbin-aspnetcore/commit/85ad73cca19ae59ff6fb94e2e1e67510bfbd4975 fixed this issue.