search

casbin / xorm-adapter

Xorm adapter for Casbin
https://github.com/casbin/casbin
Apache License 2.0
384 stars 58 forks source link

fixed save policy for concurrent #27

Closed linxing closed 4 years ago

linxing commented 4 years ago

I am not quite understand why not just CRUD the policy when save policy instead of drop table then create table?

Fix: https://github.com/casbin/xorm-adapter/issues/26

coveralls commented 4 years ago

Coverage Status

Coverage increased (+0.9%) to 67.713% when pulling ce86dd499b44cc33efdbce0ed7b5506f5f5699ae on linxing:fix-save-policy into 79a2aa54a016320eb29cf90090f642183827750b on casbin:master.

hsluoyz commented 4 years ago

This PR is reverted because of this: https://github.com/casbin/xorm-adapter/pull/28#issuecomment-642666837