eggjs / egg-security

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

refactor: use async function and support egg@2 #25

Closed dead-horse closed 7 years ago

dead-horse commented 7 years ago

https://github.com/eggjs/egg/issues/1564

codecov-io commented 7 years ago

Codecov Report

Merging #25 into master will decrease coverage by <.01%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #25      +/-   ##
==========================================
- Coverage   95.89%   95.88%   -0.01%     
==========================================
  Files          26       26              
  Lines         438      437       -1     
==========================================
- Hits          420      419       -1     
  Misses         18       18
Impacted Files Coverage Δ
lib/helper/shtml.js 94.73% <ø> (ø) :arrow_up:
lib/safe_redirect.js 100% <ø> (ø) :arrow_up:
lib/middlewares/noopen.js 100% <100%> (ø) :arrow_up:
lib/middlewares/hsts.js 100% <100%> (ø) :arrow_up:
lib/middlewares/methodnoallow.js 100% <100%> (ø) :arrow_up:
lib/middlewares/xssProtection.js 100% <100%> (ø) :arrow_up:
lib/middlewares/xframe.js 100% <100%> (ø) :arrow_up:
lib/middlewares/dta.js 100% <100%> (ø) :arrow_up:
lib/middlewares/csp.js 97.14% <100%> (-0.08%) :arrow_down:
lib/middlewares/csrf.js 100% <100%> (ø) :arrow_up:
... and 1 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 3fa64e1...a2d69d6. Read the comment docs.

dead-horse commented 7 years ago

测试 fixtures 先不改了,等以后有 codemod 了再统一变更,现在对验证 security 插件的逻辑没有影响

dead-horse commented 7 years ago

@eggjs/core 看看

dead-horse commented 7 years ago

中文注释去掉了

dead-horse commented 7 years ago

2.0.0