search

eggjs / egg-security

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

Ajax judgement #9

Closed jtyjty99999 closed 7 years ago

mention-bot commented 7 years ago

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

jtyjty99999 commented 7 years ago

cc @dead-horse @fengmk2 , related to https://github.com/eggjs/egg/issues/186 https://github.com/eggjs/egg/issues/195

codecov-io commented 7 years ago

Current coverage is 94.43% (diff: 100%)

Merging #9 into master will increase coverage by 0.02%

@@             master         #9   diff @@
==========================================
  Files            26         26          
  Lines           393        395     +2   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits            371        373     +2   
  Misses           22         22          
  Partials          0          0

Powered by Codecov. Last update 86706a3...28c6156

popomore commented 7 years ago

判断 form 就好了，只处理 form

popomore commented 7 years ago

@dead-horse 你自己改吧

jtyjty99999 commented 7 years ago

@dead-horse 按照之前的讨论这个mr还留着吗，还是直接重构成ctoken逻辑

dead-horse commented 7 years ago

我用 cookie 重构一版大家来看看吧

fengmk2 commented 7 years ago

@dead-horse 来