search

eggjs / egg-security

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

feat: cookie csrf token add appname #16

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 @jtyjty99999, @shaoshuai0102 and @fengmk2 to be potential reviewers.

codecov[bot] commented 7 years ago

Codecov Report

Merging #16 into master will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #16   +/-   ##
=======================================
  Coverage   95.62%   95.62%           
=======================================
  Files          25       25           
  Lines         411      411           
=======================================
  Hits          393      393           
  Misses         18       18
Impacted Files Coverage Δ
app/extend/context.js 100% <100%> (ø) :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 50608bf...abb7169. Read the comment docs.

fengmk2 commented 7 years ago

这样会导致那些取名很挫的应用名很尴尬，还是先忽略吧。之前的逻辑不兼容了。