FoalTS / foal

Full-featured Node.js framework 🚀
https://foalts.org/
MIT License
1.9k stars 142 forks source link

Add `getCsrfTokenFromRequest` util #1134

Closed LoicPoullain closed 2 years ago

LoicPoullain commented 2 years ago

Issue

The @UseSessions and @JWT hooks do too much stuff and so the tests are difficult to maintain and modify.

Solution and steps

Checklist

codecov-commenter commented 2 years ago

Codecov Report

Merging #1134 (9691960) into v3-0-0 (cf846d7) will increase coverage by 0.00%. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           v3-0-0    #1134   +/-   ##
=======================================
  Coverage   98.86%   98.86%           
=======================================
  Files          96       97    +1     
  Lines        1763     1764    +1     
  Branches      422      420    -2     
=======================================
+ Hits         1743     1744    +1     
  Misses         20       20           
Impacted Files Coverage Δ
...e/src/sessions/http/get-csrf-token-from-request.ts 100.00% <100.00%> (ø)
...ckages/core/src/sessions/http/use-sessions.hook.ts 98.80% <100.00%> (-0.02%) :arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.