casbin-rs / actix-casbin-auth

Casbin Actix-web access control middleware
https://github.com/casbin/casbin-rs
55 stars 15 forks source link

Fix for actix-web 4.0.1 #33

Closed thespooler closed 2 years ago

thespooler commented 2 years ago

Fix: https://github.com/casbin-rs/actix-casbin-auth/issues/27

Fix: https://github.com/casbin-rs/actix-casbin-auth/issues/31

casbin-bot commented 2 years ago

@smrpn @hackerchai @PsiACE @GopherJ please review

CLAassistant commented 2 years ago

CLA assistant check
All committers have signed the CLA.

codecov-commenter commented 2 years ago

Codecov Report

Merging #33 (b371571) into master (1a79901) will decrease coverage by 1.25%. The diff coverage is 35.71%.

@@            Coverage Diff             @@
##           master      #33      +/-   ##
==========================================
- Coverage   81.11%   79.86%   -1.26%     
==========================================
  Files           4        4              
  Lines         143      149       +6     
==========================================
+ Hits          116      119       +3     
- Misses         27       30       +3     
Impacted Files Coverage Δ
tests/test_middleware.rs 93.54% <ø> (+0.21%) :arrow_up:
tests/test_set_enforcer.rs 93.75% <ø> (+0.20%) :arrow_up:
src/middleware.rs 57.89% <35.71%> (-3.22%) :arrow_down:
tests/test_middleware_domain.rs 93.10% <0.00%> (+0.24%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 1a79901...b371571. Read the comment docs.

hsluoyz commented 2 years ago

@PsiACE