search

eggjs / egg-security

Security plugin for egg, force performance too.
MIT License
238 stars 43 forks source link

feat: use egg-path-matching to support fn #15

Closed dead-horse closed 7 years ago

mention-bot commented 7 years ago

@dead-horse, thanks for your PR! By analyzing the history of the files in this pull request, we identified @fengmk2 and @jtyjty99999 to be potential reviewers.

codecov[bot] commented 7 years ago

Codecov Report

Merging #15 into master will decrease coverage by 0.73%. The diff coverage is 85.71%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #15      +/-   ##
==========================================
- Coverage   96.35%   95.62%   -0.74%     
==========================================
  Files          25       25              
  Lines         439      411      -28     
==========================================
- Hits          423      393      -30     
- Misses         16       18       +2
Impacted Files Coverage Δ
lib/middlewares/hsts.js 100% <100%> (ø) :arrow_up:
lib/middlewares/csp.js 97.22% <100%> (-0.15%) :arrow_down:
lib/middlewares/xframe.js 100% <100%> (ø) :arrow_up:
lib/middlewares/nosniff.js 100% <100%> (ø) :arrow_up:
lib/middlewares/xssProtection.js 100% <100%> (ø) :arrow_up:
lib/middlewares/csrf.js 100% <100%> (ø) :arrow_up:
lib/middlewares/noopen.js 100% <100%> (ø) :arrow_up:
lib/utils.js 68.57% <100%> (-11.06%) :arrow_down:
app/middleware/securities.js 92.85% <60%> (-7.15%) :arrow_down:
... and 6 more

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 35c0c9d...982a95b. Read the comment docs.

shaoshuai0102 commented 7 years ago

+ egg-security@1.9.0

popomore commented 7 years ago

覆盖率下降了？

dead-horse commented 7 years ago

因为删代码了吧